[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