> Date: Sat, 8 Jun 2024 11:10:21 -0600 > From: Jeff Law <jeffreya...@gmail.com>
> >>> resource.cc: Replace calls to find_basic_block with cfgrtl > >>> BLOCK_FOR_INSN > >>> resource.cc (mark_target_live_regs): Remove check for bb not found > >>> resource.cc: Remove redundant conditionals > >> > >> I had to revert those last three patches due to PR > >> bootstrap/115284. I hope to revisit once I have a means to > >> reproduce (and fix) the underlying bug. It doesn't have to > >> be a bug with those changes per-se: IMHO the "improved" > >> lifetimes could just as well have uncovered a bug elsewhere > >> in reorg. It's still on me to resolve that situation; done. > >> I'm just glad the cause was the incidental improvements and > >> not the original bug I wanted to fix. > >> > >> There appears to be only a single supported SPARC machine in > >> cfarm: cfarm216, and I currently can't reach it due to what > >> appears to be issues at my end. I guess I'll either fix > >> that or breathe life into sparc-elf+sim. > > Or if you've got a reasonable server to use, QEMU might save you :-) > > > > Even better option. The sh4/sh4eb-linux-gnu ports with > execute/ieee/fp-cmp-5.c test. That started execution failing at -O2 > with the first patch in the series and there are very clear assembly > differences before/after your change. Meaning you can probably look at > them with just a cross compile and compare the before/after. Interesting, thanks for this. (I'd expect assembly differences, but only for slightly improved performance.) Not sure when I'll revisit the underlying problem at bootstrap/115284 though, perhaps not this week. (For context, for the gallery, for the record: the bootstrap problem there is solved.) brgds, H-P
