[PATCH] D87700: [SVE] Replace / operator in TypeSize/ElementCount with coefficientDiv

Christopher Tetreault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Sep 24 09:07:26 PDT 2020


ctetreau added a comment.

In D87700#2290260 <https://reviews.llvm.org/D87700#2290260>, @david-arm wrote:

> ... thoughts on Sander's suggestion of divideCoefficientBy?

I think the important thing is that the function has "coefficient" in the name. I prefer my name because it's shorter, but if more people prefer Sander's suggestion that's fine.


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D87700/new/

https://reviews.llvm.org/D87700



More information about the llvm-commits mailing list