[PATCH] D89592: [Inliner][NPM] Properly pass callee AAResults

[Arthur Eubanks via Phabricator via llvm-commits]

aeubanks created this revision.
aeubanks added reviewers: ychen, asbirlea, davidxl.
Herald added subscribers: llvm-commits, hiraditya, eraman.
Herald added a project: LLVM.
aeubanks requested review of this revision.

Fixes noalias-calls.ll under NPM.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D89592

Files:
  llvm/lib/Transforms/IPO/Inliner.cpp
  llvm/test/Transforms/Inline/noalias-calls.ll


Index: llvm/test/Transforms/Inline/noalias-calls.ll
===================================================================
--- llvm/test/Transforms/Inline/noalias-calls.ll
+++ llvm/test/Transforms/Inline/noalias-calls.ll
@@ -1,4 +1,5 @@
 ; RUN: opt -basic-aa -inline -enable-noalias-to-md-conversion -S < %s | FileCheck %s
+; RUN: opt -aa-pipeline=basic-aa -passes=inline -enable-noalias-to-md-conversion -S < %s | FileCheck %s
 target datalayout = "e-p:64:64:64-i1:8:8-i8:8:8-i16:16:16-i32:32:32-i64:64:64-f32:32:32-f64:64:64-v64:64:64-v128:128:128-a0:0:64-s0:64:64-f80:128:128-n8:16:32:64-S128"
 target triple = "x86_64-unknown-linux-gnu"
 
Index: llvm/lib/Transforms/IPO/Inliner.cpp
===================================================================
--- llvm/lib/Transforms/IPO/Inliner.cpp
+++ llvm/lib/Transforms/IPO/Inliner.cpp
@@ -848,7 +848,8 @@
           &FAM.getResult<BlockFrequencyAnalysis>(*(CB->getCaller())),
           &FAM.getResult<BlockFrequencyAnalysis>(Callee));
 
-      InlineResult IR = InlineFunction(*CB, IFI);
+      InlineResult IR =
+          InlineFunction(*CB, IFI, &FAM.getResult<AAManager>(*CB->getCaller()));
       if (!IR.isSuccess()) {
         Advice->recordUnsuccessfulInlining(IR);
         continue;


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D89592.298746.patch
Type: text/x-patch
Size: 1240 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20201016/5d4847c0/attachment.bin>