This is using the existing codegen so we can see the change once we custom lower ISD::{U,S}{ADD,SUB}O nodes.
Details
Details
Diff Detail
Diff Detail
Event Timeline
Comment Actions
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.
Comment Actions
Oops, I forgot to add the Differential URL in the commit.
Submitted in rL335023: [RISCV] Add tests for overflow intrinsics