[llvm] [DirectX] Legalize i8 allocas (PR #137399)

Justin Bogner via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 29 13:03:21 PDT 2025


================
@@ -81,16 +108,29 @@ static void fixI8TruncUseChain(Instruction &I,
     Value *NewInst =
         Builder.CreateBinOp(BO->getOpcode(), NewOperands[0], NewOperands[1]);
     if (auto *OBO = dyn_cast<OverflowingBinaryOperator>(&I)) {
-      if (OBO->hasNoSignedWrap())
-        cast<BinaryOperator>(NewInst)->setHasNoSignedWrap();
-      if (OBO->hasNoUnsignedWrap())
-        cast<BinaryOperator>(NewInst)->setHasNoUnsignedWrap();
+      auto *NewBO = dyn_cast<BinaryOperator>(NewInst);
----------------
bogner wrote:

Ah, I hadn't realized that `setHasNoUnsignedWrap` is private in `OverflowingBinaryOperator` and got confused. Thanks for double checking.

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


More information about the llvm-commits mailing list