[PATCH] D46399: Skip blocks with no predecessors for CFIInstrInserter verify

Petar Jovanovic via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu May 3 16:13:37 PDT 2018


petarj added a comment.

In https://reviews.llvm.org/D46399#1087133, @craig.topper wrote:

> It's not real. But bb4 is only reachable from bb2. I just wedged an empty block in the path from bb2 to bb3 in my previous case. So neither bb2 or bb4 are actually reachable from the start of the function.


I made a typo (bb3 vs bb4). I wanted to say:

and this way we reach bb3 from either:

bb0->bb1->bb3 or
bb2->bb4->bb3.

What should be a correct offset for bb3 in this case?


Repository:
  rL LLVM

https://reviews.llvm.org/D46399





More information about the llvm-commits mailing list