[llvm-dev] Textual IR value names

[David Greene via llvm-dev]

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