From d7cb643ef0fee5f479e63d339e76ec45ae033673 Mon Sep 17 00:00:00 2001
From: Enrico Granata <egranata@apple.com>
Date: Wed, 28 Jan 2015 01:41:00 +0000
Subject: [PATCH] Emit a warning the first time that one tries to resolve the
 size of a type passing a nullptr ExecutionContext - but only when it might
 actually make a difference

This should help us find these cases and act on them

llvm-svn: 227289
---
 lldb/source/Symbol/ClangASTType.cpp | 14 ++++++++++++++
 1 file changed, 14 insertions(+)

diff --git a/lldb/source/Symbol/ClangASTType.cpp b/lldb/source/Symbol/ClangASTType.cpp
index e7103fbb21e0..90958888c848 100644
--- a/lldb/source/Symbol/ClangASTType.cpp
+++ b/lldb/source/Symbol/ClangASTType.cpp
@@ -2081,6 +2081,20 @@ ClangASTType::GetBitSize (ExecutionContext *exe_ctx) const
                             return bit_size;
                     }
                 }
+                else
+                {
+                    static bool g_printed = false;
+                    StreamString s;
+                    s.Printf("warning: trying to determine the size of type ");
+                    DumpTypeDescription(&s);
+                    s.Printf("\n without a valid ExecutionContext. this is not reliable. please file a bug against LLDB.\nbacktrace:\n");
+                    Host::Backtrace(s, 10);
+                    if (!g_printed)
+                    {
+                        printf("%s\n", s.GetData());
+                        g_printed = true;
+                    }
+                }
                 // fallthrough
             default:
                 const uint32_t bit_size = m_ast->getTypeSize (qual_type);