<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Jun 20, 2014 at 4:17 PM, David Blaikie <span dir="ltr"><<a href="mailto:dblaikie@gmail.com" target="_blank" class="cremed">dblaikie@gmail.com</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">As to the other comments about whether to add a value to the enum or use a flag in createCompileUnit - for now I think I'm going to err on the side of preferring a flag in createCompileUnit.<br>
</blockquote><div><br></div><div>Sure. I'll restore the original behaviour then.</div><div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
The Clang-side support for this should obviously look a fair bit like GMLT (indeed if you put the enum value for this mode before GMLT, it should get most/all the right behavior by default (though you might have to fix up some comparisons for == GMLT to <= GMLT)). At some point we can probably simplify this even further so all the scopes are just simple line/file info without having to create any compile units, functions, lexical scopes, etc.<br>
</blockquote><div><br></div><div>Right. At the moment, the Clang-side support for this looks exactly like -gmlt. I've not needed to change anything since the code that checks for equality to DebugLineTablesOnly does not really affect loc tracking. I figured we can tweak those later (though I'll gladly change them to <=, it's only two spots that need fixing).</div>
<div> </div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Comment at: test/Transforms/LoopVectorize/X86/vectorization-remarks.ll:9<br>
@@ +8,3 @@<br>
<div class="">+; DEBUG-OUTPUT-NOT: .loc<br>
+; DEBUG-OUTPUT-NOT: {{.*}}.debug_info{{.*}}<br>
+<br>
</div>----------------<br>
You don't need the {{.*}} on either side here, FileCheck looks for subline matches by default (which is why the .loc as you've written it is correct).<br></blockquote><div><br></div><div>Done.</div><div><br></div>
<div><br></div><div>Thanks. Diego.</div></div></div></div>