<div dir="ltr"><div class="gmail_default" style>On Fri, Jan 4, 2013 at 9:48 PM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank" class="cremed">dblaikie@gmail.com</a>></span> wrote:<br>
</div><div class="gmail_extra"><div class="gmail_quote"><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">Author: dblaikie<br>
Date: Fri Jan  4 23:48:08 2013<br>
New Revision: 171604<br>
<br>
URL: <a href="http://llvm.org/viewvc/llvm-project?rev=171604&view=rev" target="_blank" class="cremed">http://llvm.org/viewvc/llvm-project?rev=171604&view=rev</a><br>
Log:<br>
Change test/CodeGenCXX/debug-info-method.cpp an IR (rather than asm) test.<br>
<br>
Referring back to the original commit (r115090) which was a frontend only test<br>
I adjusted this test to verify the frontend change that was made, to emit the<br>
protected access value in the flags metadata field.<br>
<br>
Modified:<br>
    cfe/trunk/test/CodeGenCXX/debug-info-method.cpp<br>
<br>
Modified: cfe/trunk/test/CodeGenCXX/debug-info-method.cpp<br>
URL: <a href="http://llvm.org/viewvc/llvm-project/cfe/trunk/test/CodeGenCXX/debug-info-method.cpp?rev=171604&r1=171603&r2=171604&view=diff" target="_blank" class="cremed">http://llvm.org/viewvc/llvm-project/cfe/trunk/test/CodeGenCXX/debug-info-method.cpp?rev=171604&r1=171603&r2=171604&view=diff</a><br>

==============================================================================<br>
--- cfe/trunk/test/CodeGenCXX/debug-info-method.cpp (original)<br>
+++ cfe/trunk/test/CodeGenCXX/debug-info-method.cpp Fri Jan  4 23:48:08 2013<br>
@@ -1,5 +1,5 @@<br>
-// RUN: %clang -fverbose-asm -g -S %s -o - | FileCheck %s<br>
-// CHECK: DW_ACCESS_protected<br>
+// RUN: %clang_cc1 -emit-llvm -g %s -o - | FileCheck %s<br>
+// CHECK: metadata !"_ZN1A3fooEv", {{.*}}, i32 258<br></blockquote><div><br></div><div>Oof, without your commit message, this test would be completely opaque... I mean, this isn't a problem with this commit clearly, but a problem with the way in which we handle debug info testing in IR tests that I've just never really noticed... Any ideas how we can make these types of debug info tests more understandable to future devs? My only idea is copious comments, but I feel like having some self-documenting system would be better and I just don't have any good ideas about what it would look like.<br>
</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
 class A {<br>
 protected:<br>
   int foo();<br>
<br>
<br>
_______________________________________________<br>
cfe-commits mailing list<br>
<a href="mailto:cfe-commits@cs.uiuc.edu" class="cremed">cfe-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits" target="_blank" class="cremed">http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits</a><br>
</blockquote></div><br></div></div>