[PATCH] D86431: [SVE][CodeGen] Fix up warnings in sve-split-insert/extract tests

David Sherwood via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 5 01:40:28 PST 2021


david-arm marked an inline comment as done.
david-arm added inline comments.


================
Comment at: llvm/include/llvm/Support/TypeSize.h:119
 
+  // Return the minimum value with the assumption that the count is exact.
+  // Use in places where a scalable count doesn't make sense (e.g. non-vector
----------------
fhahn wrote:
> Does this comment intentionally not use `///`? As is the documentation is missing in the rendered docs (https://llvm.org/docs/doxygen/classllvm_1_1LinearPolySize.html#a46d9c84b6f3a39f74ab8d5906c4fcf61)
No it's not intentional. I just forget to use the doxygen format sometimes. I can fix this.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D86431



More information about the llvm-commits mailing list