[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