[llvm] Remove the uaddo-only specification (PR #160392)

Simon Pilgrim via llvm-commits llvm-commits at lists.llvm.org
Thu Oct 2 03:01:09 PDT 2025


================
@@ -3405,10 +3405,7 @@ bool X86TargetLowering::shouldScalarizeBinop(SDValue VecOp) const {
 
 bool X86TargetLowering::shouldFormOverflowOp(unsigned Opcode, EVT VT,
                                              bool) const {
-  // TODO: Allow vectors?
-  if (VT.isVector())
-    return false;
-  return VT.isSimple() || !isOperationExpand(Opcode, VT);
+  return TargetLowering::shouldFormOverflowOp(Opcode, VT, true);
----------------
RKSimon wrote:

Can we drop this override entirely or is the MathUsed hack critical in come way? In which case it needs a decent comment.

https://github.com/llvm/llvm-project/pull/160392


More information about the llvm-commits mailing list