[PATCH] D38590: [ARM/AARCH64] Make test MachineBranchProb.ll more robust and re-enable for ARM/AArch64

[Balaram Makam via Phabricator via llvm-commits]

bmakam created this revision.
Herald added subscribers: kristof.beyls, mcrosier, rengolin, aemerson.

Make test robust enough to not fail due to CFG changes and re-enable for ARM/AArch64.


https://reviews.llvm.org/D38590

Files:
  test/CodeGen/Generic/MachineBranchProb.ll


Index: test/CodeGen/Generic/MachineBranchProb.ll
===================================================================
--- test/CodeGen/Generic/MachineBranchProb.ll
+++ test/CodeGen/Generic/MachineBranchProb.ll
@@ -1,16 +1,14 @@
 ; RUN: llc < %s -print-machineinstrs=expand-isel-pseudos -o /dev/null 2>&1 | FileCheck %s
 
-; ARM & AArch64 run an extra SimplifyCFG which disrupts this test.
-; UNSUPPORTED: arm,aarch64
-
 ; Hexagon runs passes that renumber the basic blocks, causing this test
 ; to fail.
 ; XFAIL: hexagon
 
 ; Bug: PR31899
 ; XFAIL: avr
 
 ; Make sure we have the correct weight attached to each successor.
+declare void @foo()
 define i32 @test2(i32 %x) nounwind uwtable readnone ssp {
 ; CHECK-LABEL: Machine code for function test2:
 entry:
@@ -29,6 +27,7 @@
 ; CHECK: Successors according to CFG: BB#1({{[0-9a-fx/= ]+}}36.36%) BB#3({{[0-9a-fx/= ]+}}63.64%)
 
 sw.bb:
+  tail call void @foo()
   br label %return
 
 sw.bb1:


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D38590.117851.patch
Type: text/x-patch
Size: 943 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20171005/9d0873ac/attachment.bin>