[PATCH] D103424: [IndVars] Don't forget value when inferring nowrap flags
Roman Lebedev via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Jun 1 02:08:27 PDT 2021
lebedev.ri added a comment.
I think i can take a look into doing this inferrence early, but thinking about it,
looks like it would be best to do that after this patch.
I do think this is *not* NFC, but i do think we can solve this more generically,
and even if we can't, the current situation does not seem reasonable.
This reduces runtime of n=512 case from https://bugs.llvm.org/show_bug.cgi?id=50384
from ~7.1s to 0.8s, i.e. an order of magnitude improvement.
I'm not sure if this actually resolves quadratic runtime problem,
it actually might, but it at least *severely* reduces it's impact.
So LGTM.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D103424/new/
https://reviews.llvm.org/D103424
More information about the llvm-commits
mailing list