[PATCH] D90462: [VE][TTI] don't advertise vregs/vops

Kazushi Marukawa via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 30 08:34:31 PDT 2020


kaz7 added inline comments.


================
Comment at: llvm/lib/Target/VE/VETargetTransformInfo.h:65
+  /// Load & Store {
+  bool isLegalMaskedLoad(Type *DataType, MaybeAlign Alignment) { return false; }
+  bool isLegalMaskedStore(Type *DataType, MaybeAlign Alignment) {
----------------
simoll wrote:
> kaz7 wrote:
> > Do we really need this and following functions?
> Not yet.
> I put them in because they are related to the notion of not advertising vops and we will define them later anyway.
If so, can you add them when those are required with some regression tests please?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D90462



More information about the llvm-commits mailing list