[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