This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGb777d354f670: [clang][DeclPrinter] Fix printing for noexcept expressions (authored by kadircet).
Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D118245/new/ https://reviews.llvm.org/D118245 Files: clang/lib/AST/DeclPrinter.cpp clang/unittests/AST/DeclPrinterTest.cpp Index: clang/unittests/AST/DeclPrinterTest.cpp =================================================================== --- clang/unittests/AST/DeclPrinterTest.cpp +++ clang/unittests/AST/DeclPrinterTest.cpp @@ -909,8 +909,7 @@ " void A(int a) noexcept(true);" "};", "A", - "void A(int a) noexcept(trueA(int a) noexcept(true)")); - // WRONG; Should be: "void A(int a) noexcept(true);" + "void A(int a) noexcept(true)")); } TEST(DeclPrinter, TestFunctionDecl_ExceptionSpecification6) { @@ -919,8 +918,7 @@ " void A(int a) noexcept(1 < 2);" "};", "A", - "void A(int a) noexcept(1 < 2A(int a) noexcept(1 < 2)")); - // WRONG; Should be: "void A(int a) noexcept(1 < 2);" + "void A(int a) noexcept(1 < 2)")); } TEST(DeclPrinter, TestFunctionDecl_ExceptionSpecification7) { @@ -930,8 +928,7 @@ " void A(int a) noexcept(N < 2);" "};", "A", - "void A(int a) noexcept(N < 2A(int a) noexcept(N < 2)")); - // WRONG; Should be: "void A(int a) noexcept(N < 2);" + "void A(int a) noexcept(N < 2)")); } TEST(DeclPrinter, TestVarDecl1) { Index: clang/lib/AST/DeclPrinter.cpp =================================================================== --- clang/lib/AST/DeclPrinter.cpp +++ clang/lib/AST/DeclPrinter.cpp @@ -731,7 +731,6 @@ FT->getNoexceptExpr()->printPretty(EOut, nullptr, SubPolicy, Indentation, "\n", &Context); EOut.flush(); - Proto += EOut.str(); Proto += ")"; } }
Index: clang/unittests/AST/DeclPrinterTest.cpp =================================================================== --- clang/unittests/AST/DeclPrinterTest.cpp +++ clang/unittests/AST/DeclPrinterTest.cpp @@ -909,8 +909,7 @@ " void A(int a) noexcept(true);" "};", "A", - "void A(int a) noexcept(trueA(int a) noexcept(true)")); - // WRONG; Should be: "void A(int a) noexcept(true);" + "void A(int a) noexcept(true)")); } TEST(DeclPrinter, TestFunctionDecl_ExceptionSpecification6) { @@ -919,8 +918,7 @@ " void A(int a) noexcept(1 < 2);" "};", "A", - "void A(int a) noexcept(1 < 2A(int a) noexcept(1 < 2)")); - // WRONG; Should be: "void A(int a) noexcept(1 < 2);" + "void A(int a) noexcept(1 < 2)")); } TEST(DeclPrinter, TestFunctionDecl_ExceptionSpecification7) { @@ -930,8 +928,7 @@ " void A(int a) noexcept(N < 2);" "};", "A", - "void A(int a) noexcept(N < 2A(int a) noexcept(N < 2)")); - // WRONG; Should be: "void A(int a) noexcept(N < 2);" + "void A(int a) noexcept(N < 2)")); } TEST(DeclPrinter, TestVarDecl1) { Index: clang/lib/AST/DeclPrinter.cpp =================================================================== --- clang/lib/AST/DeclPrinter.cpp +++ clang/lib/AST/DeclPrinter.cpp @@ -731,7 +731,6 @@ FT->getNoexceptExpr()->printPretty(EOut, nullptr, SubPolicy, Indentation, "\n", &Context); EOut.flush(); - Proto += EOut.str(); Proto += ")"; } }
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits