<div dir="ltr">Few comments:<div><br></div><div><div>+  DW_TAG_CXX_static_member = 0x104,     // Tag for C++ static data member.</div></div><div><br></div><div style>there is no such tag that I've seen? And you don't disassemble to it either.</div>
<div style><br></div><div style>Also you don't use isStaticData()...</div><div style><br></div><div style>Perhaps there's some part of the patch I'm missing?</div><div style><br></div><div style>-eric</div></div>
<div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Jan 2, 2013 at 1:38 PM, Eric Christopher <span dir="ltr"><<a href="mailto:echristo@gmail.com" target="_blank">echristo@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Sitting in my to review, but thanks for the ping I'll get to them tonight.<span class="HOEnZb"><font color="#888888"><div>
<br></div><div>-eric</div></font></span></div><div class="HOEnZb"><div class="h5"><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, Jan 2, 2013 at 1:32 PM, Robinson, Paul <span dir="ltr"><<a href="mailto:Paul.Robinson@am.sony.com" target="_blank">Paul.Robinson@am.sony.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Ping with updated patch & directly cc Eric.<br>
--paulr<br>
<br>
________________________________________<br>
From: <a href="mailto:llvm-commits-bounces@cs.uiuc.edu" target="_blank">llvm-commits-bounces@cs.uiuc.edu</a> [<a href="mailto:llvm-commits-bounces@cs.uiuc.edu" target="_blank">llvm-commits-bounces@cs.uiuc.edu</a>] on behalf of Robinson, Paul [<a href="mailto:Paul.Robinson@am.sony.com" target="_blank">Paul.Robinson@am.sony.com</a>]<br>


Sent: Tuesday, December 18, 2012 12:10 PM<br>
To: <a href="mailto:llvm-commits@cs.uiuc.edu" target="_blank">llvm-commits@cs.uiuc.edu</a><br>
Subject: [llvm-commits] [PATCH] PR14471: Debug info for static data members     (LLVM part)<br>
<div><div><br>
Define metadata interfaces for describing a static data member of a<br>
class.  Emit static data member declarations and definitions to DWARF<br>
correctly.  (Would not be hard to extend the new interface to handle<br>
all data members, rather than using the DIDerivedType hack.)<br>
<br>
LLVM part of the fix for PR14471.<br>
--paulr</div></div></blockquote></div><br></div>
</div></div></blockquote></div><br></div>