[PATCH] D42287: [GlobalISel][X86] Fixing failures after https://reviews.llvm.org/D37775

Alexander Ivchenko via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 23 08:18:21 PST 2018


aivchenk marked an inline comment as done.
aivchenk added a comment.

Hi Quentin,
I hopefully addressed all your comments

I found a bug in the initial patch, where in IncomingValueHandler:assignValueToReg we go in the new code, instead of SExt/Zext cases and we end up with G_ANYEXT, instead of G_ZEXT/G_SEXT. I made a condition stronger and added the checks whether the LocSize and ValSize are equal. For those cases we expect to have a normal flow


https://reviews.llvm.org/D42287





More information about the llvm-commits mailing list