[PATCH] D117883: [NFC] Move useSVEForFixedLengthVectors into AArch64Subtarget.h

David Sherwood via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 21 05:27:27 PST 2022


david-arm created this revision.
david-arm added reviewers: sdesmalen, peterwaller-arm, paulwalker-arm, DavidTruby.
Herald added subscribers: hiraditya, kristof.beyls, tschuett.
david-arm requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

Given how small the function is and how often it gets used it
makes more sense to live in the header file.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D117883

Files:
  llvm/lib/Target/AArch64/AArch64Subtarget.cpp
  llvm/lib/Target/AArch64/AArch64Subtarget.h


Index: llvm/lib/Target/AArch64/AArch64Subtarget.h
===================================================================
--- llvm/lib/Target/AArch64/AArch64Subtarget.h
+++ llvm/lib/Target/AArch64/AArch64Subtarget.h
@@ -680,7 +680,10 @@
     return MinSVEVectorSizeInBits;
   }
 
-  bool useSVEForFixedLengthVectors() const;
+  bool useSVEForFixedLengthVectors() const {
+    // Prefer NEON unless larger SVE registers are available.
+    return hasSVE() && getMinSVEVectorSizeInBits() >= 256;
+  }
 
   unsigned getVScaleForTuning() const { return VScaleForTuning; }
 };
Index: llvm/lib/Target/AArch64/AArch64Subtarget.cpp
===================================================================
--- llvm/lib/Target/AArch64/AArch64Subtarget.cpp
+++ llvm/lib/Target/AArch64/AArch64Subtarget.cpp
@@ -372,9 +372,4 @@
     MFI.computeMaxCallFrameSize(MF);
 }
 
-bool AArch64Subtarget::useSVEForFixedLengthVectors() const {
-  // Prefer NEON unless larger SVE registers are available.
-  return hasSVE() && getMinSVEVectorSizeInBits() >= 256;
-}
-
 bool AArch64Subtarget::useAA() const { return UseAA; }


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D117883.401960.patch
Type: text/x-patch
Size: 1091 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220121/d186ec05/attachment.bin>


More information about the llvm-commits mailing list