[PATCH] D61934: [SCEV] Add wrap flags while expanding Add
Nikita Popov via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Sun Jun 2 15:12:17 PDT 2019
nikic added a comment.
It's not entirely obvious to me that this will be correct if an existing add instruction gets reused.
I think it might make more sense to pass nowrap flags to InsertBinop and integrate more tightly there, in particular see the TODO on canGeneratePoison. That is pick an existing instruction with right nowrap flags or insert a new one, but don't reuse an instruction while assigning different flags.
CHANGES SINCE LAST ACTION
More information about the llvm-commits