commit 1c7237ae619d3d1d455f48a601e72c2795f81e40
author Richard Trieu <rtrieu@google.com> Thu Mar 31 04:18:07 2016 +0000
committer Richard Trieu <rtrieu@google.com> Thu Mar 31 04:18:07 2016 +0000
tree 709ff8e14d7d52e9748fe12a21a49e16c7e0fa32
parent 936a2b09f366267454b7cff9ffb9e76240137bf8

Fix -Wdynamic-class-memaccess to skip invalid classes.

This warning sometimes will infinitely recurse on CXXRecordDecl's from
ill-formed recursive classes that have fields of themselves.  Skip processing
these classes to prevent this from happening.
Fixes https://llvm.org/bugs/show_bug.cgi?id=27142

llvm-svn: 264991

clang/lib/Sema/SemaChecking.cpp

diff --git a/clang/lib/Sema/SemaChecking.cpp b/clang/lib/Sema/SemaChecking.cpp
index 43195bd..ee39401 100644
--- a/clang/lib/Sema/SemaChecking.cpp
+++ b/clang/lib/Sema/SemaChecking.cpp
@@ -5611,7 +5611,7 @@
 
   const CXXRecordDecl *RD = Ty->getAsCXXRecordDecl();
   RD = RD ? RD->getDefinition() : nullptr;
-  if (!RD)
+  if (!RD || RD->isInvalidDecl())
     return nullptr;
 
   if (RD->isDynamicClass())