Re: [PATCH 3/5] Make recog_op_alt consumers check the enabled attribute

2014-06-04 Thread Richard Sandiford
Thanks for the reviews, now committed Jeff Law writes: > On 05/31/14 03:15, Richard Sandiford wrote: >> As described in the covering note, it seems better to put the onus of >> checking the enabled attribute on the passes that are walking each >> alternative, like LRA does for its internal subpas

Re: [PATCH 3/5] Make recog_op_alt consumers check the enabled attribute

2014-06-03 Thread Jeff Law
On 05/31/14 03:15, Richard Sandiford wrote: As described in the covering note, it seems better to put the onus of checking the enabled attribute on the passes that are walking each alternative, like LRA does for its internal subpasses. That leads to a nicer interface in patch 4 and would make it

[PATCH 3/5] Make recog_op_alt consumers check the enabled attribute

2014-05-31 Thread Richard Sandiford
As described in the covering note, it seems better to put the onus of checking the enabled attribute on the passes that are walking each alternative, like LRA does for its internal subpasses. That leads to a nicer interface in patch 4 and would make it easier to precompute the information at build