<div dir="ltr"><br><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Dec 9, 2014 at 9:37 PM, Katya Romanova <span dir="ltr"><<a href="mailto:Katya_Romanova@playstation.sony.com" target="_blank">Katya_Romanova@playstation.sony.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><span class="">================<br>
Comment at: test/DebugInfo/incorrect-variable-debugloc1.ll:6<br>
@@ +5,3 @@<br>
+<br>
+; This is a test for BZ #21176.<br>
+; DW_OP_const <const> doesn't describe a constant value, but a value at a constant address.<br>
----------------<br>
</span><span class="">dblaikie wrote:<br>
> Usually we write this as "PR21176" (rather than "BZ #21176").<br>
</span>I will make this change in the comment before I commit. I guess, It might make the grep more consistent.<br>
I don't see any benefit in uploading one more review for this reason only.<br>
<br>
Is it OK to commit or should I wait to make sure that Fred and Eric don't have any additional comments?<br>
Katya.<br></blockquote><div><br>It's OK to commit - as you can see in Phab, I "approved" the change, which generally means you're good to commit. Other people can provide post-commit review feedback if there's more followup needed.<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<span class=""><br>
================<br>
Comment at: test/DebugInfo/incorrect-variable-debugloc1.ll:22<br>
@@ +21,3 @@<br>
+<br>
+; DWARF23: Location description: 10 0d<br>
+; DWARF4: Location description: 10 0d 9f<br>
----------------<br>
</span><span class="">dblaikie wrote:<br>
> You probably need a {{$}} at the end of this line, otherwise it'll pass even if we emit the DWARF4 form here (test this by removing the conditional in your fix and observing that this test doesn't fail where it should)<br>
</span>You are right. I'll fix it.<br>
<br>
<a href="http://reviews.llvm.org/D6568" target="_blank">http://reviews.llvm.org/D6568</a><br>
<br>
<br>
</blockquote></div><br></div></div>