[PATCH] D94874: [InstCombine] Transform abs pattern using multiplication to abs intrinsic (PR45691)

[Dávid Bolvanský via Phabricator via llvm-commits]

xbolva00 marked 2 inline comments as done.
xbolva00 added inline comments.


================
Comment at: llvm/lib/Transforms/InstCombine/InstCombineMulDivRem.cpp:375
+    Abs->takeName(&I);
+    return replaceInstUsesWith(I, Abs);
+  }
----------------
nikic wrote:
> Doesn't just `return Builder.CreateBinaryIntrinsic` work here?
void llvm::SymbolTableListTraits<ValueSubClass>::addNodeToList(ValueSubClass*) [with ValueSubClass = llvm::Instruction]: Assertion `!V->getParent() && "Value already in a container!!"' failed.



CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D94874/new/

https://reviews.llvm.org/D94874