This is a follow-up to https://reviews.llvm.org/D80707
Compile-time changes: https://llvm-compile-time-tracker.com/compare.php?from=b9496efbb9ebbf7ec76411fac4e12ff0dc7a8e06&to=4ef4b3b3cc00344aa6e3ff8691e484f1ef7bafba&stat=instructions There is a positive impact, but it's very small. No text size changes.
I've got this patch locally, it raises three errors. I'll submit the patch, the passes update once fixed and when everything lands we can finally consider landing this patch :-)
Thanks for the positive feedback!
Hrm, the committed patches showed text size changes at O3, so probably something isn't right here: https://llvm-compile-time-tracker.com/compare.php?from=5f1cad4d296a20025f0b3ea30d82b278ddb8779b&to=2296182181521c0b1803a0cd10b098fcfdab1c92&stat=size-text
Relevant files are CMakeFiles/SPASS.dir/renaming.c.o, CMakeFiles/7zip-benchmark.dir/CPP/7zip/Archive/Common/CoderMixer2MT.cpp.o
Okay, this is not related to this patch after all. I see a couple of text size flips in consecutive commits, so some (other) fairly recent change must have introduced compiler non-determinism :(