[PATCH] D155708: [AArch64][NFC] Call the API getVScaleRange directly

Allen zhong via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Jul 26 02:29:13 PDT 2023


Allen marked an inline comment as done.
Allen added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64TargetMachine.cpp:420
   if (MaxSVEVectorSize == 0)
-    MinSVEVectorSize = (MinSVEVectorSize / 128) * 128;
+    MinSVEVectorSize = MinSVEVectorSize;
   else {
----------------
DavidSpickett wrote:
> ```
> /work/open_source/lldb-cross-compile/llvm-project/llvm/lib/Target/AArch64/AArch64TargetMachine.cpp:420:22: warning: explicitly assigning value of variable of type 'unsigned int' to itself [-Wself-assign]
>     MinSVEVectorSize = MinSVEVectorSize;
>     ~~~~~~~~~~~~~~~~ ^ ~~~~~~~~~~~~~~~~
> ```
> Compiling with clang-11.
Thanks for the reminder, I already revert , and will fix this bug.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D155708



More information about the llvm-commits mailing list