<div dir="ltr"><span style="font-size:12.8px">Moving this discussion here from the review thread.</span><div style="font-size:12.8px"><br></div><div style="font-size:12.8px"><div>RIght now, we have a few passes that prefix DEBUG message with various things, so folks using -debug can tell where the debug message came from.</div><div><br></div><div>But this is not consistent, and in fact, most passes don't do it.</div><div><br>Worse, the prefixes used do not often match the DEBUG_TYPE of the pass.</div><div><br></div><div>It would be nice to fix this, and just have the DEBUG macros output the debug type in front of messages if that's what folks want.</div><div><br></div><div>Of course, it turns out you can't just make the DEBUG macro do something here, because people use it for more than just outputting messages.</div><div><br></div><div>My thought was to just bite the bullet and make a DEBUG_MSG macro that outputs the DEBUG_TYPE, and replace the appropriate existing uses of DEBUG(dbgs() <<) with it.</div><div><br></div><div>Something like</div><div>#define DEBUG_MSG_WITH_TYPE(t, x) DEBUG_WITH_TYPE(t, dbgs() << t <<":" << x;)</div><div>#define DEBUG_MSG(x) DEBUG_MSG_WITH_TYPE(DEBUG_TYPE, x)<br></div><div><br></div><div>Suggestions, however, welcome!</div><div><br></div><div>---------- Forwarded message ----------<br></div><div><div class="gmail_quote">From: <b class="gmail_sendername">Renato Golin</b> <span dir="ltr"><<a href="mailto:renato.golin@linaro.org" target="_blank">renato.golin@linaro.org</a>></span><br>Date: Fri, Oct 30, 2015 at 10:15 AM<br>Subject: Re: [PATCH] D13259: LLE 6/6: Add LoopLoadElimination pass<br>To: <a href="mailto:anemet@apple.com" target="_blank">anemet@apple.com</a>, <a href="mailto:hfinkel@anl.gov" target="_blank">hfinkel@anl.gov</a>, <a href="mailto:dberlin@dberlin.org" target="_blank">dberlin@dberlin.org</a><br>Cc: <a href="mailto:mssimpso@codeaurora.org" target="_blank">mssimpso@codeaurora.org</a>, <a href="mailto:sanjoy@playingwithpointers.com" target="_blank">sanjoy@playingwithpointers.com</a>, <a href="mailto:llvm-commits@lists.llvm.org" target="_blank">llvm-commits@lists.llvm.org</a><br><br><br>rengolin added a comment.<br><br>In <a href="http://reviews.llvm.org/D13259#278362" rel="noreferrer" target="_blank">http://reviews.llvm.org/D13259#278362</a>, @dberlin wrote:<br><br>> I would suggest rather than prefix *anything*, anywhere, that if we want<br>>  prefixes, we just have the DEBUG macro output the debug type in front of<br>>  the message ;-)<br><br><br>Sounds good to me.<br><br><br><a href="http://reviews.llvm.org/D13259" rel="noreferrer" target="_blank">http://reviews.llvm.org/D13259</a></div></div></div></div>