On 7/10/24 11:42 AM, Adhemerval Zanella Netto wrote:
On 02/07/24 16:40, Jeff Law wrote:
[ Actually attaching the patch this time... ]
The pre-commit testing showed that making ext-dce only active at -O2 and above
would require minor edits to the tests. In some cases we had specified -O1 in
the test or specified no optimization level at all. Those need to be bumped to
-O2. In one test we had one set of dg-options overriding another.
The other approach that could have been taken would be to drop the -On
argument, add an explicit -fext-dce and add dg-skip-if options. I originally
thought that was going to be way to go, but the dg-skip-if aspect was going to
get ugly as things like interaction between unrolling, peeling and -ftracer
would have to be accounted for and would likely need semi-regular adjustment.
Changes since V2:
Testsuite changes to deal with pass only being enabled at -O2 or
higher.
Hi Jeff,
It seems that this commit has triggered an issue on 32 bit ARM [1]
FAIL: gcc.dg/plugin/must-tail-call-1.c -fplugin=./must_tail_call_plugin.so
(internal compiler error: in df_refs_verify, at df-scan.cc:4013)
Should I open a bug report for this?
[1]
https://git-us.linaro.org/toolchain/ci/interesting-commits.git/plain/gcc/sha1/98914f9eba5f19d3eb93fbce8726b5264631cba0/tcwg_bootstrap_check/master-arm-check_bootstrap/details.txt
Wasn't this pre-existing and just the line number of the fault changed?
jeff
