[PATCH] D134282: [CGP] Add generic TargetLowering::shouldAlignPointerArgs() implementation

Eli Friedman via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 28 10:55:26 PDT 2022


efriedma added inline comments.


================
Comment at: llvm/include/llvm/CodeGen/TargetLowering.h:1925
+  virtual bool shouldUpdatePointerArgAlignment(const CallInst *CI,
+                                               const Value *Arg,
+                                               unsigned &MinSize,
----------------
It looks like the argument "Arg" is now unused?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D134282



More information about the llvm-commits mailing list