Make the argument to getFeatureMask and isMultiversionedFunction a constant reference.

Author: labrinea

llvm/include/llvm/Analysis/TargetTransformInfo.h

@@ -1872,10 +1872,10 @@ class TargetTransformInfo {
 
   /// Returns a bitmask constructed from the target-features or fmv-features
   /// metadata of a function.
-  uint64_t getFeatureMask(Function &F) const;
+  uint64_t getFeatureMask(const Function &F) const;
 
   /// Returns true if this is an instance of a function with multiple versions.
-  bool isMultiversionedFunction(Function &F) const;
+  bool isMultiversionedFunction(const Function &F) const;
 
   /// \return The maximum number of function arguments the target supports.
   unsigned getMaxNumArgs() const;
@@ -2319,8 +2319,8 @@ class TargetTransformInfo::Concept {
   virtual VPLegalization
   getVPLegalizationStrategy(const VPIntrinsic &PI) const = 0;
   virtual bool hasArmWideBranch(bool Thumb) const = 0;
-  virtual uint64_t getFeatureMask(Function &F) const = 0;
-  virtual bool isMultiversionedFunction(Function &F) const = 0;
+  virtual uint64_t getFeatureMask(const Function &F) const = 0;
+  virtual bool isMultiversionedFunction(const Function &F) const = 0;
   virtual unsigned getMaxNumArgs() const = 0;
   virtual unsigned getNumBytesToPadGlobalArray(unsigned Size,
                                                Type *ArrayType) const = 0;
@@ -3153,11 +3153,11 @@ class TargetTransformInfo::Model final : public TargetTransformInfo::Concept {
     return Impl.hasArmWideBranch(Thumb);
   }
 
-  uint64_t getFeatureMask(Function &F) const override {
+  uint64_t getFeatureMask(const Function &F) const override {
     return Impl.getFeatureMask(F);
   }
 
-  bool isMultiversionedFunction(Function &F) const override {
+  bool isMultiversionedFunction(const Function &F) const override {
     return Impl.isMultiversionedFunction(F);
   }
 

llvm/include/llvm/Analysis/TargetTransformInfoImpl.h

@@ -1039,9 +1039,9 @@ class TargetTransformInfoImplBase {
 
   bool hasArmWideBranch(bool) const { return false; }
 
-  uint64_t getFeatureMask(Function &F) const { return 0; }
+  uint64_t getFeatureMask(const Function &F) const { return 0; }
 
-  bool isMultiversionedFunction(Function &F) const { return false; }
+  bool isMultiversionedFunction(const Function &F) const { return false; }
 
   unsigned getMaxNumArgs() const { return UINT_MAX; }
 

llvm/lib/Analysis/TargetTransformInfo.cpp

@@ -1383,11 +1383,11 @@ bool TargetTransformInfo::hasArmWideBranch(bool Thumb) const {
   return TTIImpl->hasArmWideBranch(Thumb);
 }
 
-uint64_t TargetTransformInfo::getFeatureMask(Function &F) const {
+uint64_t TargetTransformInfo::getFeatureMask(const Function &F) const {
   return TTIImpl->getFeatureMask(F);
 }
 
-bool TargetTransformInfo::isMultiversionedFunction(Function &F) const {
+bool TargetTransformInfo::isMultiversionedFunction(const Function &F) const {
   return TTIImpl->isMultiversionedFunction(F);
 }
 

llvm/lib/Target/AArch64/AArch64TargetTransformInfo.cpp

@@ -249,7 +249,7 @@ static bool hasPossibleIncompatibleOps(const Function *F) {
   return false;
 }
 
-uint64_t AArch64TTIImpl::getFeatureMask(Function &F) const {
+uint64_t AArch64TTIImpl::getFeatureMask(const Function &F) const {
   StringRef AttributeStr =
       isMultiversionedFunction(F) ? "fmv-features" : "target-features";
   StringRef FeatureStr = F.getFnAttribute(AttributeStr).getValueAsString();
@@ -258,7 +258,7 @@ uint64_t AArch64TTIImpl::getFeatureMask(Function &F) const {
   return AArch64::getFMVPriority(Features);
 }
 
-bool AArch64TTIImpl::isMultiversionedFunction(Function &F) const {
+bool AArch64TTIImpl::isMultiversionedFunction(const Function &F) const {
   return F.hasFnAttribute("fmv-features");
 }
 

llvm/lib/Target/AArch64/AArch64TargetTransformInfo.h

@@ -89,9 +89,9 @@ class AArch64TTIImpl : public BasicTTIImplBase<AArch64TTIImpl> {
   unsigned getInlineCallPenalty(const Function *F, const CallBase &Call,
                                 unsigned DefaultCallPenalty) const;
 
-  uint64_t getFeatureMask(Function &F) const;
+  uint64_t getFeatureMask(const Function &F) const;
 
-  bool isMultiversionedFunction(Function &F) const;
+  bool isMultiversionedFunction(const Function &F) const;
 
   /// \name Scalar TTI Implementations
   /// @{