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