[llvm] 24c993d - [Attributor][FIX] Update unit test after API change

Johannes Doerfert via llvm-commits llvm-commits at lists.llvm.org
Thu Jul 21 23:05:54 PDT 2022


Author: Johannes Doerfert
Date: 2022-07-22T01:05:33-05:00
New Revision: 24c993dab6bec662bfae6ed6dfaf22c6a18cc0ce

URL: https://github.com/llvm/llvm-project/commit/24c993dab6bec662bfae6ed6dfaf22c6a18cc0ce
DIFF: https://github.com/llvm/llvm-project/commit/24c993dab6bec662bfae6ed6dfaf22c6a18cc0ce.diff

LOG: [Attributor][FIX] Update unit test after API change

Added: 
    

Modified: 
    llvm/unittests/Transforms/IPO/AttributorTest.cpp

Removed: 
    


################################################################################
diff  --git a/llvm/unittests/Transforms/IPO/AttributorTest.cpp b/llvm/unittests/Transforms/IPO/AttributorTest.cpp
index 18c7730708b4..0d1b04f642c8 100644
--- a/llvm/unittests/Transforms/IPO/AttributorTest.cpp
+++ b/llvm/unittests/Transforms/IPO/AttributorTest.cpp
@@ -187,7 +187,6 @@ TEST_F(AttributorTestBase, AAReachabilityTest) {
   F7AA.canReach(A, F7FirstCB, F3);
   F7AA.canReach(A, F7FirstCB, F4);
   F9AA.instructionCanReach(A, F9FirstInst, F3);
-  F9AA.instructionCanReach(A, F9SecondInst, F3, false);
   F9AA.instructionCanReach(A, F9FirstInst, F4);
 
   A.run();
@@ -203,10 +202,7 @@ TEST_F(AttributorTestBase, AAReachabilityTest) {
   ASSERT_TRUE(F6AA.canReach(A, F4));
 
   // The second instruction of F9 can't reach the first call.
-  ASSERT_FALSE(F9AA.instructionCanReach(A, F9SecondInst, F3, false));
-  // TODO: Without lifetime limiting callback this query does actually not make
-  //       much sense. "Anything" is reachable from the caller of func10.
-  ASSERT_TRUE(F9AA.instructionCanReach(A, F9SecondInst, F3, true));
+  ASSERT_FALSE(F9AA.instructionCanReach(A, F9SecondInst, F3));
 
   // The first instruction of F9 can reach the first call.
   ASSERT_TRUE(F9AA.instructionCanReach(A, F9FirstInst, F3));


        


More information about the llvm-commits mailing list