[PATCH] D116851: [MemoryBuiltins] Add field for alignment argument

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

xbolva00 added inline comments.


================
Comment at: llvm/lib/Transforms/InstCombine/InstCombineCalls.cpp:2594
+  if (AlignOpC && AlignOpC->getValue().ult(llvm::Value::MaximumAlignment) &&
       isKnownNonZero(Call.getOperand(1), DL, 0, &AC, &Call, &DT)) {
+    uint64_t AlignmentVal = AlignOpC->getZExtValue();
----------------
Works fine currently but we should replace Call.getOperand(1) with some more general API like

getAllocSizeArg or so.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D116851