[PATCH] D103991: [SCEV] Move mustprogress based no-self-wrap logic so it applies to all exit conditions
Philip Reames via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Nov 19 08:27:17 PST 2021
reames added a comment.
In D103991#3142766 <https://reviews.llvm.org/D103991#3142766>, @nikic wrote:
> Also no difference. I suspect that we're seeing some kind of second order effect here, not an issue in the code itself.
I agree and am going to revert the two speculative changes as they just complicate the code.
I don't really know what else to do here on the original regression. It sure seems like inferring flags is simply making some other piece of code slower. As things stand, I plan to leave the original code in tree as the regression is small, but I find that result unsatisfying.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D103991/new/
https://reviews.llvm.org/D103991
More information about the llvm-commits
mailing list