<div dir="ltr">Hi Paul,<div><br></div><div>Looking much better. Few comments:</div><div><br></div><div><div>+// RUN: %clangxx -O0 -g %s -c -o %t.o</div><div>+// RUN: %clangxx %t.o -o %t.out</div><div>+// RUN: %test_debuginfo %s %t.out</div>

<div>+</div><div><br></div><div>Can probably do the compile and link in one line if we're not using the temporary .o file.</div><div><br></div><div>+// DEBUGGER: delete breakpoints</div><div>+// DEBUGGER: break main</div>

<div>+// DEBUGGER: r</div><div>+// DEBUGGER: n</div><div>+// DEBUGGER: p instance_C</div><div>+// CHECK: $1 = {static a = 4, static b = {{.*}}, static c = 15, d = {{.*}}}</div><div>+</div></div><div><br></div><div>Might also want to ptype C as well?</div>

<div><br></div><div><div>+class C</div><div>+{</div></div><div><br></div><div style>Formatting in the testcase if you don't mind.</div><div style><br></div><div style>Otherwise, much nicer! Same comments as the llvm one :)</div>
<div style><br></div><div style>-eric</div><div class="gmail_extra"><br><br><div class="gmail_quote">On Tue, Jan 8, 2013 at 5:16 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:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">New and improved, refreshed and invigorated patch for PR14471/14734.<br>

<br>
- CGDebugInfo::CollectRecordFields() decomposed with 3 new helpers.<br>
- The "fieldNo - 1" is gone, thanks to the refactoring.<br>
- Commentary about the field counting added.<br>
- getStaticDataMemberDeclaration() checks the decl before doing the search.<br>
I hope that was everything!<br>
<br>
--paulr</blockquote></div><br></div></div>