On Sat, Aug 18, 2012 at 1:23 AM, Uros Bizjak <ubiz...@gmail.com> wrote: > On Sat, Aug 18, 2012 at 10:14 AM, Uros Bizjak <ubiz...@gmail.com> wrote: > >> After discussion with Oleg, it looks that it is enough to prevent >> wrong registers in the output of the (multi-output) insn pattern. As >> far as inputs are concerned, combine already handles limited reload >> classes in the right way. The problem with x86 is, that reload tried >> to fix output operand of the multi-output ins, where scheduler already >> moved load of ax register before this insn. >> >> Version 2 of the patch now handles only output operands. Also, >> handling of empty constraints was fixed. > > ... but here we fail testcase from PR46843... so we HAVE to check > input operands. >
If this is very critical, can you add a target hook to decide whether to check input/output operands? -- H.J.
