[PATCH] D107305: [GlobalISel] Combine G_ASSERT_*EXT + G_LOAD -> G_S/ZEXTLOAD

Amara Emerson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 2 13:34:38 PDT 2021


aemerson added a comment.

I don't think this is right. G_ASSERT_SEXT is supposed to only provide information for optimizations about the bits of the source. Using it to justify transforming the source seems the wrong place to fix it. We should be emitting the right kind of load in call lowering.


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

https://reviews.llvm.org/D107305



More information about the llvm-commits mailing list