[llvm] 7fb9de2 - [StackSafety,NFC] Fix tests in debug

Vitaly Buka via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 6 20:46:51 PDT 2020


Author: Vitaly Buka
Date: 2020-08-06T20:46:39-07:00
New Revision: 7fb9de2c6f575d807e614ed6021736ba701876a8

URL: https://github.com/llvm/llvm-project/commit/7fb9de2c6f575d807e614ed6021736ba701876a8
DIFF: https://github.com/llvm/llvm-project/commit/7fb9de2c6f575d807e614ed6021736ba701876a8.diff

LOG: [StackSafety,NFC] Fix tests in debug

Added: 
    

Modified: 
    llvm/lib/Analysis/StackSafetyAnalysis.cpp
    llvm/test/Analysis/StackSafetyAnalysis/ipa.ll

Removed: 
    


################################################################################
diff  --git a/llvm/lib/Analysis/StackSafetyAnalysis.cpp b/llvm/lib/Analysis/StackSafetyAnalysis.cpp
index 56039d72b0c5..042062a449c9 100644
--- a/llvm/lib/Analysis/StackSafetyAnalysis.cpp
+++ b/llvm/lib/Analysis/StackSafetyAnalysis.cpp
@@ -617,7 +617,6 @@ GlobalValueSummary *getGlobalValueSummary(const ModuleSummaryIndex *Index,
   auto VI = Index->getValueInfo(ValueGUID);
   if (!VI || VI.getSummaryList().empty())
     return nullptr;
-  assert(VI.getSummaryList().size() == 1);
   auto &Summary = VI.getSummaryList()[0];
   return Summary.get();
 }

diff  --git a/llvm/test/Analysis/StackSafetyAnalysis/ipa.ll b/llvm/test/Analysis/StackSafetyAnalysis/ipa.ll
index 21a47b2b6da3..f42745dabd03 100644
--- a/llvm/test/Analysis/StackSafetyAnalysis/ipa.ll
+++ b/llvm/test/Analysis/StackSafetyAnalysis/ipa.ll
@@ -48,13 +48,13 @@
 ; RUN:  -r %t.summ0.bc,TwoArgumentsOOBBoth,px \
 ; RUN:  -r %t.summ0.bc,TwoArgumentsOOBOne,px \
 ; RUN:  -r %t.summ0.bc,TwoArgumentsOOBOther,px \
-; RUN:  -r %t.summ0.bc,Weak,px \
+; RUN:  -r %t.summ0.bc,Weak,x \
 ; RUN:  -r %t.summ0.bc,Write1, \
-; RUN:  -r %t.summ0.bc,Write1DiffModule,px \
+; RUN:  -r %t.summ0.bc,Write1DiffModule,x \
 ; RUN:  -r %t.summ0.bc,Write1Module0,px \
-; RUN:  -r %t.summ0.bc,Write1Private,px \
-; RUN:  -r %t.summ0.bc,Write1SameModule,px \
-; RUN:  -r %t.summ0.bc,Write1Weak,px \
+; RUN:  -r %t.summ0.bc,Write1Private,x \
+; RUN:  -r %t.summ0.bc,Write1SameModule,x \
+; RUN:  -r %t.summ0.bc,Write1Weak,x \
 ; RUN:  -r %t.summ0.bc,Write4_2, \
 ; RUN:  -r %t.summ0.bc,Write4, \
 ; RUN:  -r %t.summ0.bc,Write8, \
@@ -69,10 +69,10 @@
 ; RUN:  -r %t.summ1.bc,RecursiveWithOffset,px \
 ; RUN:  -r %t.summ1.bc,ReturnAlloca,px \
 ; RUN:  -r %t.summ1.bc,ReturnDependent,px \
-; RUN:  -r %t.summ1.bc,Weak,px \
+; RUN:  -r %t.summ1.bc,Weak,x \
 ; RUN:  -r %t.summ1.bc,Write1,px \
 ; RUN:  -r %t.summ1.bc,Write1DiffModule,px \
-; RUN:  -r %t.summ1.bc,Write1Module0,px \
+; RUN:  -r %t.summ1.bc,Write1Module0,x \
 ; RUN:  -r %t.summ1.bc,Write1Private,px \
 ; RUN:  -r %t.summ1.bc,Write1SameModule,px \
 ; RUN:  -r %t.summ1.bc,Write1Weak,px \
@@ -115,13 +115,13 @@
 ; RUN:  -r %t.summ0.bc,TwoArgumentsOOBBoth,px \
 ; RUN:  -r %t.summ0.bc,TwoArgumentsOOBOne,px \
 ; RUN:  -r %t.summ0.bc,TwoArgumentsOOBOther,px \
-; RUN:  -r %t.summ0.bc,Weak,px \
+; RUN:  -r %t.summ0.bc,Weak,x \
 ; RUN:  -r %t.summ0.bc,Write1, \
-; RUN:  -r %t.summ0.bc,Write1DiffModule,px \
+; RUN:  -r %t.summ0.bc,Write1DiffModule,x \
 ; RUN:  -r %t.summ0.bc,Write1Module0,px \
-; RUN:  -r %t.summ0.bc,Write1Private,px \
-; RUN:  -r %t.summ0.bc,Write1SameModule,px \
-; RUN:  -r %t.summ0.bc,Write1Weak,px \
+; RUN:  -r %t.summ0.bc,Write1Private,x \
+; RUN:  -r %t.summ0.bc,Write1SameModule,x \
+; RUN:  -r %t.summ0.bc,Write1Weak,x \
 ; RUN:  -r %t.summ0.bc,Write4_2, \
 ; RUN:  -r %t.summ0.bc,Write4, \
 ; RUN:  -r %t.summ0.bc,Write8, \
@@ -136,10 +136,10 @@
 ; RUN:  -r %t.summ1.bc,RecursiveWithOffset,px \
 ; RUN:  -r %t.summ1.bc,ReturnAlloca,px \
 ; RUN:  -r %t.summ1.bc,ReturnDependent,px \
-; RUN:  -r %t.summ1.bc,Weak,px \
+; RUN:  -r %t.summ1.bc,Weak,x \
 ; RUN:  -r %t.summ1.bc,Write1,px \
 ; RUN:  -r %t.summ1.bc,Write1DiffModule,px \
-; RUN:  -r %t.summ1.bc,Write1Module0,px \
+; RUN:  -r %t.summ1.bc,Write1Module0,x \
 ; RUN:  -r %t.summ1.bc,Write1Private,px \
 ; RUN:  -r %t.summ1.bc,Write1SameModule,px \
 ; RUN:  -r %t.summ1.bc,Write1Weak,px \
@@ -554,7 +554,7 @@ define void @TestCrossModuleWeak() #0 {
 ; CHECK-NEXT: allocas uses:
 ; LOCAL-NEXT: x[1]: empty-set, @Write1Weak(arg0, [0,1)){{$}}
 ; NOLTO-NEXT: x[1]: [1,2), @Write1Weak(arg0, [0,1)){{$}}
-; LTO-NEXT: x[1]: [-1,0), @Write1Weak(arg0, [0,1)){{$}}
+; LTO-NEXT: x[1]: full-set, @Write1Weak(arg0, [0,1)){{$}}
 ; CHECK-EMPTY:
 entry:
   %x = alloca i8, align 4


        


More information about the llvm-commits mailing list