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

David Sherwood via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 23 08:49:48 PDT 2020


david-arm added a comment.

I think that introducing a halve() function makes sense and I'll replace div(2) with halve(), unless someone strongly objects? I don't have strong feelings about the name to be honest, I'm personally happy enough to stick with coefficientDiv. @ctetreau any thoughts on Sander's suggestion of divideCoefficientBy?


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

https://reviews.llvm.org/D87700



More information about the llvm-commits mailing list