On 10/6/22 01:06, Stefan Schulze Frielinghaus wrote:
On Wed, Oct 05, 2022 at 08:48:13PM -0600, Jeff Law via Gcc-patches wrote:
On 10/4/22 05:28, Stefan Schulze Frielinghaus via Gcc-patches wrote:
For BImode get_narrowest_mode evaluates to QImode but BImode < QImode.
Thus FOR_EACH_MODE_UNTIL never reaches BImode and iterates until OImode
for which no wider mode exists so we end up with VOIDmode and fail.
Fixed by adding a size guard so we effectively skip BImode.
Bootstrap and regtest are currently running on x64. Assuming they pass
ok for mainline?
gcc/ChangeLog:
PR rtl-optimization/107088
* cselib.cc (new_cselib_val): Skip BImode while keeping track of
subvalue relations.
OK. And FWIW, this fixes the various failures I saw in my tester due to the
cselib patches.
Thanks for testing, too! Out of curiosity which target is your tester?
I gave it a try on x64 and AArch64 for which bootstrap went fine and
regtest showed no difference, and of course, for s390x regtest went for
the better.
It's got at least one representative from most of the targets supported
by gcc.
http://law-sandy.freeddns.org:8080 will get you to the jenkins instance.
Jeff
