Commit 2ae2b4ec authored by Bjorn Pettersson's avatar Bjorn Pettersson
Browse files

[ConstantHoisting][test] Adjust lit test to hopefully work on windows (2nd attempt) 

Problem seem to be that on windows it says
  "Assertion failed: ...."
while one linux we get
  "Assertion ... failed."

Anyway, this fixup also limits the test case to x86_64-linux to
avoid buildbot failures due to such problems.
parent caeb636e

llvm/test/Transforms/ConstantHoisting/X86/pr52689-not-all-uses-rebased.ll

+4 −2
Original line number Diff line number Diff line
@@ -2,6 +2,10 @@ 


; REQUIRES: asserts



; Matching assertion strings is not easy as they might differ on different

; platforms. So limit this to x86_64-linux.

; REQUIRES: x86_64-linux



; This is a reproducer for https://github.com/llvm/llvm-project/issues/52689

;

; opt: ../lib/Transforms/Scalar/ConstantHoisting.cpp:919: bool llvm::ConstantHoistingPass::emitBaseConstants(llvm::GlobalVariable *): Assertion `UsesNum == (ReBasesNum + NotRebasedNum) && "Not all uses are rebased"' failed.
@@ -9,8 +13,6 @@ 
; CHECK: Assertion

; CHECK-SAME: UsesNum == (ReBasesNum + NotRebasedNum)

; CHECK-SAME: Not all uses are rebased

; CHECK-SAME: failed.





@g_77 = external global [5 x i32]