hassnaaHamdi wrote: > Thanks, I'm a big +1 on this. Predication is one of the many overloaded terms > in the vectorizer and it would be good to disambiguate it. > > Should we also rename `TargetTransformInfo::preferPredicateOverEpilogue` > whilst we're at it?
Happy that you like it. In terms of `TargetTransformInfo::preferPredicateOverEpilogue`, I will publish another PR to rename the `ScalarEpilogueLowering` enum and I will include that function also. I just tried to minimize this PR as mush as possible :'D https://github.com/llvm/llvm-project/pull/191803 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
