<div dir="ltr"><br><br><div class="gmail_quote"><div dir="ltr">On Wed, Feb 1, 2017 at 4:56 PM David L. Jones 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">dlj added inline comments.<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
================<br class="gmail_msg">
Comment at: test/DebugInfo/Generic/gmlt.test:2<br class="gmail_msg">
 ; REQUIRES: object-emission<br class="gmail_msg">
 ; RUN: %llc_dwarf -O0 -filetype=obj < %S/../Inputs/gmlt.ll | llvm-dwarfdump - | FileCheck %S/../Inputs/gmlt.ll<br class="gmail_msg">
+; RUN: %llc_dwarf -O0 -filetype=obj < %S/../Inputs/gmlt_profiling.ll | llvm-dwarfdump - | FileCheck %S/../Inputs/gmlt_profiling.ll<br class="gmail_msg">
----------------<br class="gmail_msg">
dblaikie wrote:<br class="gmail_msg">
> This line should probably be in gmlt.ll (both gmlt.ll and gmlt_profiling.ll would move from test/DebugInfo/Inputs to test/DebugInfo/Generic)<br class="gmail_msg">
Moving this file to Generic/ means that other tests (notably, under X86/) now use  files not within their Inputs, or within their parents' Inputs. Was that the intent of this comment?<br class="gmail_msg"></blockquote><div><br></div><div>Ah - hadn't looked at that. If it's worth keeping the distinct tests sharing non-trivial IR, best to switch it back to the way it was. If the IR's trivial enough, might be easier for each test to have its own (makes the tests easier to read, etc).<br><br>Ah, I see - it's so that it's easier to filter the different tests, but they're still pretty legible because all FileCheck logic is in the IR file.<br><br>Not sure why there's an X86 version of this test if there's also a generic version, though.<br><br>The intent wasn't to break any other tests, no - it looked like the test was generic (still looks like it - so I'm a bit confused) so I wasn't sure why it would be referring to a separate IR file (still not sure why, really).<br><br>*reads/stares more*<br><br>I guess it's because the darwin case is XFAIL'd so it had to be in a separate file... <br><br>Looks like maybe this has all got a bit confused. I'll take a look.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
<br class="gmail_msg">
<br class="gmail_msg">
<a href="https://reviews.llvm.org/D29203" rel="noreferrer" class="gmail_msg" target="_blank">https://reviews.llvm.org/D29203</a><br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
<br class="gmail_msg">
</blockquote></div></div>