only inline when featurebits are equal

Author: chavandres

llvm/lib/Target/SystemZ/SystemZTargetTransformInfo.cpp

@@ -431,9 +431,8 @@ bool SystemZTTIImpl::areInlineCompatible(const Function *Caller,
   const FeatureBitset &CalleeBits =
       TM.getSubtargetImpl(*Callee)->getFeatureBits();
 
-  // Check that target features from the callee are subset or
-  // equal to the caller's features.
-  return (CalleeBits == CallerBits) || (CalleeBits < CallerBits);
+  // Check that target features from the callee are equal to the caller's features.
+  return CallerBits == CalleeBits;
 }
 
 unsigned SystemZTTIImpl::getNumberOfRegisters(unsigned ClassID) const {

llvm/test/Transforms/Inline/SystemZ/inline-target-attr.ll

@@ -1,6 +1,6 @@
 ; RUN: opt < %s -mtriple=s390x-linux-gnu -S -passes=inline | FileCheck %s
 ; RUN: opt < %s -mtriple=s390x-linux-gnu -S -passes='cgscc(inline)' | FileCheck %s
-; Check that we only inline when we have compatible target attributes.
+; Check that we only inline when we have equal target attributes.
 
 define i32 @foo() #0 {
 entry:
@@ -17,31 +17,23 @@ entry:
   %call = call i32 @foo()
   ret i32 %call
 ; CHECK-LABEL: bar
-; CHECK: call i32 (...) @baz()
+; CHECK: call i32 @foo()
 }
 
 define i32 @qux() #0 {
 entry:
-  %call = call i32 @bar()
+  %call = call i32 @foo()
   ret i32 %call
 ; CHECK-LABEL: qux
-; CHECK: call i32 @bar()
+; CHECK: call i32 (...) @baz()
 }
 
 define i32 @quux() #2 {
 entry:
   %call = call i32 @bar()
   ret i32 %call
 ; CHECK-LABEL: quux
-; CHECK: call i32 (...) @baz()
-}
-
-define i32 @foobar() #1 {
-entry:
-  %call = call i32 @bar()
-  ret i32 %call
-; CHECK-LABEL: foobar
-; CHECK: call i32 (...) @baz()
+; CHECK: call i32 @bar()
 }