[PATCH] D113294: [IR] Remove unbounded as possible value for vscale_range minimum

Paul Walker via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 6 07:51:45 PST 2021


paulwalker-arm accepted this revision.
paulwalker-arm added a comment.
This revision is now accepted and ready to land.

One minor issue but otherwise looks good.



================
Comment at: clang/lib/Basic/Targets/AArch64.cpp:482
+    return std::pair<unsigned, unsigned>(
+        LangOpts.VScaleMin ? LangOpts.VScaleMin : 1, 16);
+
----------------
This part is no longer needed because to get here you already know `LangOpts.VScaleMin==0 && LangOpts.VScaleMax==0`. 


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

https://reviews.llvm.org/D113294



More information about the llvm-commits mailing list