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.
