<div dir="ltr"><div class="gmail_extra"><div class="gmail_quote">On Fri, Jan 3, 2014 at 2:03 PM, Nico Weber <span dir="ltr"><<a href="mailto:thakis@chromium.org" target="_blank">thakis@chromium.org</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr"><div class="im">On Fri, Jan 3, 2014 at 10:15 AM, Nico Weber <span dir="ltr"><<a href="mailto:thakis@chromium.org" target="_blank">thakis@chromium.org</a>></span> wrote:<br>
</div><div class="gmail_extra"><div class="gmail_quote"><div class="im">
<blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Ok, sounds like it's not clear if folks want to have dump methods behind #ifdef !NDEBUG. Maybe we don't have to resolve that part in this thread<div>

<br></div><div>Is there any opposition to replacing the attributes on just the dump methods with LLVM_DUMP_METHOD? That's a subset of Chandler's change and I think also what Alp's diff is.</div>
<div><br></div></div></blockquote></div><div>…which is effectively Alp's patch I suppose. So maybe we could land that for now and then discuss whether to put dump() methods in !NDEBUG at some other point?</div></div>
</div></div></blockquote><div><br></div><div>I'd like to actually see the patch rather than talk in the hypothetical. Maybe I misunderstood, but I thought it also moved the attribute to the definition? I don't understand why that's relevant yet (which is why maybe I've misunderstood what was happening). I've asked what the bug was that this addressed. My reading of the spec for the attribute seems to indicate that there is no difference....</div>
<div><br></div></div></div></div>