[PATCH] [FastISel][X86] Assign the correct register class to folded operations
Pete Cooper
peter_cooper at apple.com
Mon Jun 1 13:14:18 PDT 2015
In http://reviews.llvm.org/D6262#181993, @loladiro wrote:
> @pete looks like you fixed this one as well (though this time in http://reviews.llvm.org/rL236644). Maybe I'll just wait for somebody else to fix it next time ;). Would you mind taking a look here and seeing if this is still interesting?
Yep, looks like my fix handled this one too. I don't think this patch will handle more cases than mine. If you have a strong preference for your solution over mine, then i'm happy to see the code changed though. There's no perfect way to fix this bug so i'm happy to see the best possible solution committed.
http://reviews.llvm.org/D6262
EMAIL PREFERENCES
http://reviews.llvm.org/settings/panel/emailpreferences/
More information about the llvm-commits
mailing list