<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Tue, Aug 1, 2017 at 7:33 AM Paul Robinson via Phabricator <<a href="mailto:reviews@reviews.llvm.org">reviews@reviews.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">probinson added inline comments.<br>
<br>
<br>
================<br>
Comment at: test/Transforms/GlobalOpt/static-global-boolean-dwarf.ll:2<br>
+; RUN: opt -f %s -globalopt -o %t<br>
+; llvm-dwarfdump %t | FileCheck %s<br>
+<br>
----------------<br>
It would be more usual to pipe the output of opt directly to llvm-dwarfdump, unless opt can't do that.<br>
(It helps the tests run faster to avoid writing files unless you really have to.)<br></blockquote><div><br>Yep (& opt can do it ("-o -")). Also usually the input file is streamed in ("< %s") this avoids the input filename becoming part of the output and possibly accidentally messing with CHECKs on the output.<br> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br>
<br>
================<br>
Comment at: test/Transforms/GlobalOpt/static-global-boolean-dwarf.ll:128<br>
+<br>
+; CHECK:   DW_AT_name [DW_FORM_strp] {{.*}} "foo"<br>
+; CHECK-NEXT:   DW_AT_type<br>
----------------<br>
These checks verify that something named "foo" has debug info of some kind, but without verifying any details.  I'd rather see verification of the location expression, and possibly the type, given that you are planning to change those in the future to describe the shrunk variable.<br>
<br>
<br>
<a href="https://reviews.llvm.org/D35994" rel="noreferrer" target="_blank">https://reviews.llvm.org/D35994</a><br>
<br>
<br>
<br>
</blockquote></div></div>