[PATCH] D83523: MachineSink: permit sinking into INLINEASM_BR indirect targets

Nick Desaulniers via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 13 11:28:23 PDT 2020


nickdesaulniers added a comment.

In D83523#2147995 <https://reviews.llvm.org/D83523#2147995>, @nickdesaulniers wrote:

> In D83523#2146520 <https://reviews.llvm.org/D83523#2146520>, @void wrote:
>
> > I think this might be a better fix:
>
>
> It seems that none of those fix the provided test case. :-(


Ah, right, because the test case is still testing machinesink. Let me change it to twoaddressinstruction and retest.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D83523





More information about the llvm-commits mailing list