[PATCH] D45439: [IRCE] Use NUW flag for indvar
Max Kazantsev via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Apr 13 03:50:31 PDT 2018
mkazantsev added a comment.
Interesting... I've made a dummy patch which lifts the restricton on `nsw` in cases when we are not dealing with `eq/ne`, and it passed 5000 complex fuzzed tests (which usually fail if there is some obvious bug). I also don't see a place where we would really need the no wrap other than `eq/ne`.
@samparker do you have a chance to prepare a patch that lifts this restriction? I can do it as well, but I'm a bit loaded atm. If we can get rid of no-wrap requirements at all, it will be a way better than using `nuw`. :)
https://reviews.llvm.org/D45439
More information about the llvm-commits
mailing list