[PATCH] [FastIsel][X86] Fix invalid register replacement for bool args

Keno Fischer kfischer at college.harvard.edu
Thu Nov 13 18:42:03 PST 2014


Unfortunately, this patch isn't quite correct. Since the OutVals[i] isn't updated it'll bail out in the next loop, since it thinks the argument is an i1, which isn't legal. However, even after fixing that the trunc is now in the value map, causing it to get put into FuncInfo.ValueMap and hence subsequently isel'd (which we don't want, because we already emit the code for it). Any idea how to fix that?

http://reviews.llvm.org/D6245






More information about the llvm-commits mailing list