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

Cullen Rhodes via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 6 08:04:23 PST 2021


c-rhodes added inline comments.


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

Ah of course, I'll fix before committing, cheers!


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

https://reviews.llvm.org/D113294



More information about the llvm-commits mailing list