[PATCH] D71320: [IR] Split out target specific intrinsic enums into separate headers

Eli Friedman via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Dec 10 17:14:17 PST 2019


efriedma added inline comments.


================
Comment at: llvm/include/llvm/Analysis/TargetTransformInfo.h:759
                     Type *Ty) const;
-  int getIntImmCost(Intrinsic::ID IID, unsigned Idx, const APInt &Imm,
-                    Type *Ty) const;
+  int getIntImmCostIntrin(Intrinsic::ID IID, unsigned Idx, const APInt &Imm,
+                          Type *Ty) const;
----------------
For the sake of anyone downstream, it would be better to rename both overloads.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D71320





More information about the cfe-commits mailing list