commit ddd01cec0e4ecec0ca2f6e02c38281ee562fcd8d
author Richard Trieu <rtrieu@google.com> Mon Jun 09 22:53:25 2014 +0000
committer Richard Trieu <rtrieu@google.com> Mon Jun 09 22:53:25 2014 +0000
tree 37c95111a4e9ec687e3be04eb7f4e149d693c0e2
parent a23043cb9c1ef021a9cf05cd62cce76cd03c0ba2

Removing an "if (this == nullptr)" check from two print methods.  The condition
will never be true in a well-defined context.  The checking for null pointers
has been moved into the caller logic so it does not rely on undefined behavior.

llvm-svn: 210498

clang/lib/Sema/SemaExceptionSpec.cpp

diff --git a/clang/lib/Sema/SemaExceptionSpec.cpp b/clang/lib/Sema/SemaExceptionSpec.cpp
index 24d8222..6eccb9f 100644
--- a/clang/lib/Sema/SemaExceptionSpec.cpp
+++ b/clang/lib/Sema/SemaExceptionSpec.cpp
@@ -283,6 +283,7 @@
 
   case EST_ComputedNoexcept:
     OS << "noexcept(";
+    assert(OldProto->getNoexceptExpr() != nullptr && "Expected non-null Expr");
     OldProto->getNoexceptExpr()->printPretty(OS, nullptr, getPrintingPolicy());
     OS << ")";
     break;