[PATCH] D89088: [MBP] Add whole chain to BlockFilterSet instead of individual BB

Tue Oct 20 14:26:42 PDT 2020

dmajor added a subscriber: jmorse.
dmajor added a comment.

Adding @jmorse in case D83046 <https://reviews.llvm.org/D83046> is related.

I managed to retrieve a linkrepro tarball from our CI builder, but it's 4GB and pretty cumbersome.

The dump of the affected machine function is: https://paste.mozilla.org/g44ZAPGr. We hit the assertion when processing `bb.2`. Its only predecessor `bb.1` was skipped with `ignoring unvisited pred MBB: 1`, so in the end `NumVisited` is still zero.

(Aside: is it normal for `bb.2` to come before `bb.0` in the `dump()` output? I'm not familiar with the conventions in this area.)

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D89088/new/

https://reviews.llvm.org/D89088