Denis Chertykov schrieb: > 2011/4/17 Denis Chertykov <cherty...@gmail.com>: >> 2011/4/15 Georg-Johann Lay <a...@gjlay.de>: >>> Finally, I exposed alternative #3 of the insns to the register >>> allocator, because it is not possible to distinguish between >>> overlapping or non-overlapping regs, and #3 does not need a scratch. >>> >>> Ran C-testsuite with no regressions. >> Are you encountered any difference in code size ? > > I'm ask about code size because the IRA pass isn't work with > `scratch:MODE' at all. > This lead to bad/wrong register allocation in IRA pass. > The reload pass will correct such a wrong allocation, but reload can't > generate optimal code. (reload generate correct code). > Because of that, may be you right and may be better to have > (clobber (match_operand....)) instead of (clobber (match_scratch...)).
So the conclusion is not to commit this patch and that the one-liner already installed is sufficient to fix the ICE? > > Denis. >
