[PATCH] D47926: [RISCV] Add tests for overflow intrinsics

Alex Bradbury via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 14 06:58:47 PDT 2018


asb accepted this revision.
asb added a comment.
This revision is now accepted and ready to land.

Thanks! Looks good to me, but two minor suggestions:

- arith-with-overflow.ll might be a better name
- Add the nounwind attribute to the function definitions, so we still get clean assembly output when cfi directives are emitted.


https://reviews.llvm.org/D47926





More information about the llvm-commits mailing list