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

Reply via email to