<div dir="ltr">Hi Paul,<div><br></div><div>I think this is going to be OK. I've got a couple comments though:</div><div><br></div><div><div>-                          layout.getFieldOffset(fieldNo), tunit, RecordTy);</div>

<div>+                          layout.getFieldOffset(fieldNo - 1), tunit, RecordTy);</div><div><br></div></div><div><div>a) There are a few other places that use fieldNo and since you're not updating them it'd be nice to have some comments.</div>
<div style>b) Instead of inlining all the code for CollectRecordStaticVars how about outlining it into a couple of static functions?</div><div style>c) Will this handle something like this:</div><div style><br></div><div style>
class A {</div><div style>  static int a;</div><div style>};</div><div style><br></div><div style>A a;</div><div style><br></div><div style>where the variable A::a isn't defined in the file (but likely is somewhere else).</div>
<div style><br></div><div style>-eric</div>
<div><br></div></div></div><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 w/ updated patch & directly cc Eric.<br>
Also should fix PR14734.  Added a debuginfo-test.<br>
--paulr<br>
<br>
________________________________________<br>
From: <a href="mailto:cfe-commits-bounces@cs.uiuc.edu">cfe-commits-bounces@cs.uiuc.edu</a> [<a href="mailto:cfe-commits-bounces@cs.uiuc.edu">cfe-commits-bounces@cs.uiuc.edu</a>] on behalf of Robinson, Paul [<a href="mailto:Paul.Robinson@am.sony.com">Paul.Robinson@am.sony.com</a>]<br>

Sent: Tuesday, December 18, 2012 12:10 PM<br>
To: <a href="mailto:cfe-commits@cs.uiuc.edu">cfe-commits@cs.uiuc.edu</a><br>
Subject: [cfe-commits] [PATCH] PR14471: Debug info for static data members      (Clang part)<br>
<div class="HOEnZb"><div class="h5"><br>
Collect both normal and static data members of a class in source<br>
order.  Describe static data members to the metadata using the new<br>
LLVM interfaces.<br>
<br>
Clang part of the fix for PR14471.<br>
--paulr</div></div></blockquote></div><br></div>