diff --git a/llvm/test/tools/llvm-diff/callbr.ll b/llvm/test/tools/llvm-diff/callbr.ll new file mode 100644 --- /dev/null +++ b/llvm/test/tools/llvm-diff/callbr.ll @@ -0,0 +1,39 @@ +; RUN: not llvm-diff %s %s 2>&1 | FileCheck %s + +define void @foo() { +entry: + callbr void asm sideeffect "", "X,X,~{dirflag},~{fpsr},~{flags}"(i8* blockaddress(@foo, %return), i8* blockaddress(@foo, %t_no)) + to label %asm.fallthrough [label %return, label %t_no] + +asm.fallthrough: + br label %return + +t_no: + br label %return + +return: + ret void +} + +; CHECK: in function bar: +; CHECK-NOT: in function foo: +; CHECK-NEXT: in block %entry: +; CHECK-NEXT: > callbr void asm sideeffect "", "X,X,~{dirflag},~{fpsr},~{flags}"(i8* blockaddress(@foo, %t_no), i8* blockaddress(@foo, %return)) +; CHECK-NEXT: to label %asm.fallthrough [label %return, label %t_no] +; CHECK-NEXT: < callbr void asm sideeffect "", "X,X,~{dirflag},~{fpsr},~{flags}"(i8* blockaddress(@foo, %t_no), i8* blockaddress(@foo, %return)) +; CHECK-NEXT: to label %asm.fallthrough [label %return, label %t_no] + +define void @bar() { +entry: + callbr void asm sideeffect "", "X,X,~{dirflag},~{fpsr},~{flags}"(i8* blockaddress(@foo, %t_no), i8* blockaddress(@foo, %return)) + to label %asm.fallthrough [label %return, label %t_no] + +asm.fallthrough: + br label %return + +t_no: + br label %return + +return: + ret void +} diff --git a/llvm/tools/llvm-diff/DifferenceEngine.cpp b/llvm/tools/llvm-diff/DifferenceEngine.cpp --- a/llvm/tools/llvm-diff/DifferenceEngine.cpp +++ b/llvm/tools/llvm-diff/DifferenceEngine.cpp @@ -284,6 +284,27 @@ } return false; + } else if (isa(L)) { + CallBrInst &LI = cast(*L); + CallBrInst &RI = cast(*R); + if (LI.getNumIndirectDests() != RI.getNumIndirectDests()) { + if (Complain) + Engine.log("callbr # of indirect destinations differ"); + return true; + } + + // Perform the "try unify" step so that we can equate the indirect + // destinations before checking the call site. + for (unsigned i = 0; i < LI.getNumIndirectDests(); i++) + tryUnify(LI.getIndirectDest(i), RI.getIndirectDest(i)); + + if (diffCallSites(LI, RI, Complain)) + return true; + + if (TryUnify) + tryUnify(LI.getDefaultDest(), RI.getDefaultDest()); + return false; + } else if (isa(L)) { BranchInst *LI = cast(L); BranchInst *RI = cast(R);