[AlwaysInliner] Respect noinline call site attribute

[xbolva00]

Summary

always_inline foo() { }

bar () {

noinline foo();
}

We should prefer call site attribute over attribute on decl. This is fix for AlwaysInliner, similar fix is needed for normal Inliner (follow up).

Related to https://reviews.llvm.org/D119061

[xbolva00]

Added one more test for noinline / alwaysinline combo.

llvm/lib/Transforms/IPO/AlwaysInliner.cpp

for (Function &F : M) {
     if (F.isPresplitCoroutine())
       continue;
 
     if (!F.isDeclaration() && isInlineViable(F).isSuccess()) {
       Calls.clear();
 
       for (User *U : F.users())
         if (auto *CB = dyn_cast<CallBase>(U))
-          if (CB->getCalledFunction() == &F &&
+          if (CB->getCalledFunction() == &F && !CB->isNoInline() &&
               CB->hasFnAttr(Attribute::AlwaysInline))
             Calls.insert(CB);
 
       for (CallBase *CB : Calls) {
         Function *Caller = CB->getCaller();
         OptimizationRemarkEmitter ORE(Caller);
         auto OIC = shouldInline(
             *CB,
InlineCost AlwaysInlinerLegacyPass::getInlineCost(CallBase &CB) {
   // So we won't inline the coroutine function if it have not been unsplited
   if (Callee->isPresplitCoroutine())
     return InlineCost::getNever("unsplited coroutine call");
 
   // FIXME: We shouldn't even get here for declarations.
   if (Callee->isDeclaration())
     return InlineCost::getNever("no definition");
 
+  if (CB.isNoInline())
+    return InlineCost::getNever("noinline call site attribute");
+
   if (!CB.hasFnAttr(Attribute::AlwaysInline))
     return InlineCost::getNever("no alwaysinline attribute");
 
   auto IsViable = isInlineViable(*Callee);
   if (!IsViable.isSuccess())
     return InlineCost::getNever(IsViable.getFailureReason());
 
   return InlineCost::getAlways("always inliner");
 }

llvm/test/Transforms/Inline/always-inline.ll

; CHECK: define void @inner14
   ret void
 }
 
 define void @outer14() {
 ; CHECK: call void @inner14
   call void @inner14()
   ret void
 }
+
+define internal i32 @inner15() {
+; CHECK: @inner15(
+  ret i32 1
+}
+
+define i32 @outer15() {
+; CHECK-LABEL: @outer15(
+; CHECK: call
+
+   %r = call i32 @inner15() noinline
+   ret i32 %r
+}
+
+define internal i32 @inner16() alwaysinline {
+; CHECK: @inner16(
+  ret i32 1
+}
+
+define i32 @outer16() {
+; CHECK-LABEL: @outer16(
+; CHECK: call
+
+   %r = call i32 @inner16() noinline
+   ret i32 %r
+}
+
+define i32 @inner17() alwaysinline {
+; CHECK: @inner17(
+  ret i32 1
+}
+
+define i32 @outer17() {
+; CHECK-LABEL: @outer17(
+; CHECK: call
+
+   %r = call i32 @inner17() noinline
+   ret i32 %r
+}