[PATCH] D121641: MemoryBuiltins: also check function definition for allocalign

[Nikita Popov via Phabricator via llvm-commits]

nikic added inline comments.


================
Comment at: llvm/lib/Analysis/MemoryBuiltins.cpp:347
+                                                &AllocAlignParam))) {
+    return V->getOperand(AllocAlignParam - 1);
   }
----------------
Maybe we should have a helper function on CallBase for this whole logic? See `CallBase::getReturnedArgOperand()` which implements exactly the same but with Attribute::Returned instead of Attribute::AllocAlign.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D121641