[PATCH] D79031: [GlobalISel] Assign the correct debug location when combining G_SEXT/G_ANYEXT/G_ZEXT

Davide Italiano via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 28 14:02:44 PDT 2020


davide added a comment.

In D79031#2008718 <https://reviews.llvm.org/D79031#2008718>, @vsk wrote:

> I'm not sure the G_SEXT case is covered -- could you add a sext(trunc x) pattern to the MIR?


Good point, I'll revert that bit & do that in a subsequent commit.

> Also, I think you can delete the IR contents of `@main`, those shouldn't be needed to construct the MIR.

That didn't work.


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

https://reviews.llvm.org/D79031





More information about the llvm-commits mailing list