[PATCH] D80510: Update BFI when handling inlined landing / eh pad

Mircea Trofin via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 1 15:43:35 PDT 2020


mtrofin added a comment.

In D80510#2067484 <https://reviews.llvm.org/D80510#2067484>, @hhb wrote:

> In D80510#2067446 <https://reviews.llvm.org/D80510#2067446>, @hhb wrote:
>
> > D75341 <https://reviews.llvm.org/D75341> and comments there seem highly related.
>
>
> Actually that change supposes to fix this kind of non-deterministic. Not sure why it does not work as intended.
>
> We may still want to copy BFI when split BBs. But other concerns should have been solved by CallbackVH...


Huh.. added davidxl for more background, too, he remembered something along the lines of this non-determinism being addressed somehow (probably via the CallbackVH)


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D80510





More information about the llvm-commits mailing list