[PATCH] D78191: [GlobalISel] Fix invalid combine of unmerge(merge) with intermediate cast

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 15 12:03:37 PDT 2020


arsenm added a comment.

In D78191#1984249 <https://reviews.llvm.org/D78191#1984249>, @arsenm wrote:

> In D78191#1984088 <https://reviews.llvm.org/D78191#1984088>, @gargaroff wrote:
>
> > Unfortunately this caused the build bots to fail due to the sanitizers flagging undefined behavior in the AMDGPUInstructionSelector and I have reverted the commit for now. @arsenm Do you think this patch should wait until the bug in the InstructionSelector is fixed? Or should this patch include a fix? I have not looked into the problem yet, so I cannot say what goes wrong exactly.
>
>
> I'm assuming this is hitting a null dereference due to the missing 192-bit register class


Try after 588bd7be366620d2319d349f7665b503d7840f45 <https://reviews.llvm.org/rG588bd7be366620d2319d349f7665b503d7840f45>


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D78191





More information about the llvm-commits mailing list