SixWeining accepted this revision. SixWeining added a comment. This revision is now accepted and ready to land.
LGTM except 2 nits. And let's wait others. ================ Comment at: clang/lib/Basic/Targets/LoongArch.cpp:174 + const std::vector<std::string> &FeaturesVec) const { + + if (getTriple().getArch() == llvm::Triple::loongarch64) ---------------- Delete blank line. ================ Comment at: clang/lib/Sema/SemaChecking.cpp:3677 + CallExpr *TheCall) { + + switch (BuiltinID) { ---------------- Delete blank line. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D137316/new/ https://reviews.llvm.org/D137316 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits