[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