[PATCH] D103702: [AArch64][SVE] Wire up vscale_range attribute to SVE min/max vector queries

Bradley Smith via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 14 04:46:30 PDT 2021


bsmith added inline comments.


================
Comment at: llvm/lib/Target/AArch64/AArch64Subtarget.cpp:386-388
+  if (MaxSVEVectorSizeInBits == 0)
+    return (MinSVEVectorSizeInBits / 128) * 128;
+  return (std::min(MinSVEVectorSizeInBits, MaxSVEVectorSizeInBits) / 128) * 128;
----------------
paulwalker-arm wrote:
> Based on my other comments I'm wondering if we just got to the point where both these functions need to be moved into `AArch64TargetMachine.cpp` with `AArch64Subtarget` having simple accessors.
I don't think we can go quite this far as AArch64TargetMachine is not function specific like the attributes are. I think we can move all of the logic of resolving between the options and the attribute, but the storage/accessors for the resulting value needs to live in the subtarget.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D103702



More information about the llvm-commits mailing list