[llvm-dev] Textual IR value names

via llvm-dev
Thu, 10 Jan 2019 20:23:54 -0000

> -----Original Message-----
> From: llvm-dev [mailto:llvm-dev-bounces at lists.llvm.org] On Behalf Of
> Cranmer, Joshua via llvm-dev
> Sent: Thursday, January 10, 2019 3:14 PM
> To: David Greene; Craig Topper
> Cc: llvm-dev at lists.llvm.org
> Subject: Re: [llvm-dev] Textual IR value names
>
> 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.

Sounds like you want to file a feature-request bugzilla. And it's more
or less the same thing that David Greene seems to want.
--paulr

>
> -----Original Message-----
> From: llvm-dev [mailto:llvm-dev-bounces at lists.llvm.org] On Behalf Of
David
> Greene via llvm-dev
> Sent: Thursday, January 10, 2019 15:05
> To: Craig Topper <craig.topper at gmail.com>
> Cc: llvm-dev at lists.llvm.org
> Subject: Re: [llvm-dev] Textual IR value names
>
> Craig Topper via llvm-dev <llvm-dev at lists.llvm.org> writes:
>
> > The names are dropped to save memory when a release build of the
> > compiler is being used. This is what you probably want on a release
> > compiler you intend to ship since it should be faster. The NDEBUG
> > check is an easy way to tell the difference between release and debug
> > builds. People probably don't want to have to remember to set an
> > additional cmake option to make a release compiler faster.
>
> 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.
>
> 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.
>
>                             -David