<div dir="ltr">I'm confused.<div>AFAIK, we normally do *not* prefix the messages explicitly (i see this pass does it in some places but not others, that should be changed to be not prefixed, IMHO).<div><br></div><div>This pass has DEBUG_TYPE set to loop-load-elim at the top, so if you use debug-only, you won't get them unless you specify debug-only=loop-load-elim.</div><div><br></div><div>I would suggest rather than prefix *anything*, anywhere, that if we want prefixes, we just have the DEBUG macro output the debug type in front of the message ;-)</div><div><br></div><div>Then it's automatic, not confusing, and clear where it came from.</div><div><br></div><div><br></div><div><br></div><div class="gmail_extra"><br><div class="gmail_quote">On Fri, Oct 30, 2015 at 9:01 AM, Renato Golin <span dir="ltr"><<a href="mailto:renato.golin@linaro.org" target="_blank">renato.golin@linaro.org</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">rengolin added a comment.<br>
<br>
While working on top of this patch, I noticed that not all DEBUG() messages are prefixed with "LLE:", which may confuse people using -debug-only=.<br>
<br>
<br>
<a href="http://reviews.llvm.org/D13259" rel="noreferrer" target="_blank">http://reviews.llvm.org/D13259</a><br>
<br>
<br>
<br>
</blockquote></div><br></div></div></div>