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 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
