Hello Uros, On 23 Dec 17:46, Uros Bizjak wrote: > This "round_expand_predicate" is the predicate substitution I was > referred to in the review of 5/8. Please use it also in insn patterns, > perhaps renamed as "round_predicate"
This is drawback of substs. We bind given subst attribute to given subst strictly. So, this guy: +(define_subst_attr "round_expand_predicate" "round_expand" "nonimmediate_operand" "register_operand") is binded to "round_expand" (second argument of definition) subst and to it only. That is way name is "round_expand...", it reflects subst it relates to. For rest substs I'll introduce dedicated attributes. -- Thanks, K