On Wed, Nov 25, 2020 at 07:32:44PM +0800, Hongtao Liu wrote: > Update patch: > 1. ix86_expand_special_args_builtin is used for expanding mask load > intrinsics, this function will always convert the constant mask > operands into reg. So for the situation of all-ones mask, keep this > constant, and also change the mask operand predicate(of corresponding > expander) to register_or_constm1_operand. > 2. Delete last_arg_constant which is not used in > ix86_expand_special_args_builtin(maybe should be in a separate patch?)
Yes, please make it a separate patch, it should go in first and should just drop last_arg_constant from that function plus the reindentation. Then post the PR97642 incremental to that. Thanks. Jakub