[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