Re: [PATCH 0/5] Cache recog_op_alt by insn code, take 2

2014-06-04 Thread Richard Sandiford
Jeff Law writes: > On 05/31/14 03:02, Richard Sandiford wrote: >> A second difference was that preprocess_constraints skips disabled >> alternatives while LRA's setup_operand_alternative doesn't; LRA just >> checks for disabled alternatives when walking the array instead. >> That should make no di

Re: [PATCH 0/5] Cache recog_op_alt by insn code, take 2

2014-06-03 Thread Jeff Law
On 05/31/14 03:02, Richard Sandiford wrote: Sorry Jeff, while working on the follow-on LRA patch I came across a couple of problems, so I need another round on this. It happens, particularly for conceptual changes like this. I don't consider that a problem at all -- we're already in agreement o

[PATCH 0/5] Cache recog_op_alt by insn code, take 2

2014-05-31 Thread Richard Sandiford
Sorry Jeff, while working on the follow-on LRA patch I came across a couple of problems, so I need another round on this. First of all, I mentioned doing a follow-on patch to make LRA and recog use the same cache for operand_alternatives. I hadn't realised until I wrote that patch that there's a