[llvm] [AMDGPU][TargetLowering] Allow forming overflow op if it is legal (PR #156266)
via llvm-commits
llvm-commits at lists.llvm.org
Tue Sep 23 04:09:06 PDT 2025
================
@@ -3455,6 +3455,10 @@ class LLVM_ABI TargetLoweringBase {
/// matching of other patterns.
virtual bool shouldFormOverflowOp(unsigned Opcode, EVT VT,
bool MathUsed) const {
+ // Form it if it is legal.
+ if (isOperationLegal(Opcode, VT))
+ return true;
+
----------------
AZero13 wrote:
That should be for another PR because that's causes so many code changes on its own outside the scope of this.
Also I believe math used just means the addition itself is used?
Then again I didn't write this logic.
https://github.com/llvm/llvm-project/pull/156266
More information about the llvm-commits
mailing list