[PATCH] D41944: [LLVM][IR][LIT] support of 'no-overflow' flag for sdiv\udiv instructions

Matthias Braun via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jan 11 09:52:57 PST 2018


MatzeB added a comment.
Herald added a subscriber: niosHD.

- Without any change in the backend (not even an abort) this will simply miscompile the no nof version on most targets.
- With the way you are modeling the new flag, means that existing bitcode/.ll files will change semantics when read with newer compilers. I'm not sure that is a good idea for this, in any way at the very least you have to provide AutoUpgrade logic for that.


Repository:
  rL LLVM

https://reviews.llvm.org/D41944





More information about the llvm-commits mailing list