[PATCH] D155900: [TTI][NFCI] Introduce two new target transform hooks

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 20 15:46:41 PDT 2023


arsenm added inline comments.


================
Comment at: llvm/include/llvm/Analysis/TargetTransformInfoImpl.h:822
+    if (!ParamAS.hasAttributes() && !ArgAS.hasAttributes())
+      return true;
+    if (ParamAS.getByValType() != ArgAS.getByValType())
----------------
arsenm wrote:
> Should also check inreg matches
Also careful of arbitrary string attributes?


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

https://reviews.llvm.org/D155900



More information about the llvm-commits mailing list