[PATCH] D153570: [CostModel][SLP] Use getGEPCost AccessTy in more places

Luke Lau via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 29 08:47:34 PDT 2023


luke added inline comments.


================
Comment at: llvm/include/llvm/Analysis/TargetTransformInfo.h:27
 #include "llvm/IR/InstrTypes.h"
+#include "llvm/IR/Instructions.h"
 #include "llvm/IR/PassManager.h"
----------------
ABataev wrote:
> Do you really need this include in the header file?
I believe I had to include to use `GetElementPtrInst`, but now that I think about it maybe it's possible to just use `GEPOperator` instead.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D153570



More information about the llvm-commits mailing list