[PATCH] D36649: [X86][ARM][TargetLowering] Add SrcVT to isExtractSubvectorCheap

Simon Pilgrim via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sun Aug 13 06:14:22 PDT 2017


RKSimon accepted this revision.
RKSimon added a comment.
This revision is now accepted and ready to land.

LGTM - one minor



================
Comment at: include/llvm/Target/TargetLowering.h:2179
   /// and only the target knows which lowering is cheap.
-  virtual bool isExtractSubvectorCheap(EVT ResVT, unsigned Index) const {
+  virtual bool isExtractSubvectorCheap(EVT ResVT, EVT SrcVT,
+                                       unsigned Index) const {
----------------
Update comment to describe ResVT and SrcVT


https://reviews.llvm.org/D36649





More information about the llvm-commits mailing list