search for: fdiscard

And the clang behavior can be controlled with -fdiscard-value-names/-fno-discard-value-names ~Craig On Wed, Jan 9, 2019 at 2:16 PM Davide Italiano via llvm-dev < llvm-dev at lists.llvm.org> wrote: > On Wed, Jan 9, 2019 at 2:12 PM David Greene via llvm-dev > <llvm-dev at lists.llvm.org> wrote: > > > > I like my LLVM IR t...

...ou 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 -fdiscard-value-names/-fno-discard-value-names was added about a year ago https://reviews.llvm.org/D42887 not sure if there is more discussion about the asserts behavior discussed there. ~Craig On Wed, Jan 9, 2019 at 2:25 PM David Greene <dag at cray.com> wrote: > Craig Topper via llvm-dev <...

I like my LLVM IR text to have nice value names, e.g. %add = add ... %mul = mul ... And this all works well if the build has asserts enabled. If the build does not have asserts enabled, it's not so nice: %1 = add ... %2 = mul ... I understand the use for obfuscating names, but the choice to make this dependent on whether or not asserts are enabled seems odd to me. At the very least

...more code. [r324803](http://reviews.llvm.org/rL324803). ## Clang commits * The `trivial_abi` attribute has been added. This indicates the annotated C++ class/struct/union to be passed or returned using the C ABI for the underlying type. [r324269](http://reviews.llvm.org/rL324269). * The flags `-fdiscard-value-names` and `-fno-discard-value-names` were added to allow user control over whether names should be discarded in IR output. [r324498](http://reviews.llvm.org/rL324498). * A TemplateInstantiationObserver class was added. This is used in order to implement the [TempLight](http://plc.inf.elte.h...