On Wed, Aug 13, 2014 at 2:14 PM, Kirill Yukhin <kirill.yuk...@gmail.com> wrote:
>> In a couple of places, <MODE_SIZE> checks are changed to GET_MODE_SIZE >> (GET_MODE (operands[0])) and in a similar way, <MODE>mode checks were >> changed to GET_MODE (operands[0]). The previous checks are more >> efficient and are preferred. Is there a reason you need to check >> operands[0] dynamically? > > I agree, runtime tests are worse. But this subst attributes are used > in patterns which actually do not contain mode iterators at all. IMO, two different subst patterns should be used in this case, one for patterns with and one for patterns w/o mode iterators. Uros.
