<div dir="ltr"><div>Having it by default for -S -emit-llvm seems like a good default to me.</div><div>Is it enough to make clang's behavior independent of the build settings?</div><div><br></div><div>-- </div><div>Mehdi</div><div><br></div><br><div class="gmail_quote"><div dir="ltr">On Thu, Jan 10, 2019 at 1:14 PM James Y Knight via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex"><div dir="ltr">+1! I didn't realize that you could even change that in a release build, but I concur that it should be enabled by default at least when emitting textual asm. The efficiency rationale for having it default off doesn't really seem to apply to that case.<br></div><br><div class="gmail_quote"><div dir="ltr">On Thu, Jan 10, 2019 at 3:14 PM Cranmer, Joshua via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-style:solid;border-left-color:rgb(204,204,204);padding-left:1ex">While I understand the value of turning it off, it is somewhat annoying when you're trying to run a clang -S -emit-llvm to discover that you're not getting any value names. I would suggest that, irrespective of the kind of build you use, -S -emit-llvm should cause the value names to be defaulted to on as a principle of least surprise to the user.<br>
<br>
-----Original Message-----<br>
From: llvm-dev [mailto:<a href="mailto:llvm-dev-bounces@lists.llvm.org" target="_blank">llvm-dev-bounces@lists.llvm.org</a>] On Behalf Of David Greene via llvm-dev<br>
Sent: Thursday, January 10, 2019 15:05<br>
To: Craig Topper <<a href="mailto:craig.topper@gmail.com" target="_blank">craig.topper@gmail.com</a>><br>
Cc: <a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
Subject: Re: [llvm-dev] Textual IR value names<br>
<br>
Craig Topper via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> writes:<br>
<br>
> The names are dropped to save memory when a release build of the <br>
> compiler is being used. This is what you probably want on a release <br>
> compiler you intend to ship since it should be faster. The NDEBUG <br>
> check is an easy way to tell the difference between release and debug <br>
> builds. People probably don't want to have to remember to set an <br>
> additional cmake option to make a release compiler faster.<br>
<br>
The CMake option could override the current behavior.  Then users wouldn't be forced to remember to set an option to make release compilers faster.<br>
<br>
In the end it's not a huge deal, I just found keying off asserts to be quite surprising and it took some time for me to figure out what was going on.<br>
<br>
                            -David<br>
_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
</blockquote></div>
_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
</blockquote></div></div>