[PATCH] D122926: [X86][CostModel] Strip bitcasts when looking for store GEP operand

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Apr 7 03:54:26 PDT 2022


RKSimon added inline comments.


================
Comment at: llvm/lib/Target/X86/X86TargetTransformInfo.cpp:3910
     }
     return TTI::TCC_Basic;
   }
----------------
I wonder whether this needs inverting - and we default stores to a cost of TTI::TCC_Basic * 2 UNLESS after stripping we find all indices are constant?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D122926



More information about the llvm-commits mailing list