[PATCH] D30339: [libcxxabi] Disable calls to fprintf when building for baremetal targets in release mode

Eric Fiselier via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 24 12:30:02 PST 2017


EricWF added a comment.

In https://reviews.llvm.org/D30339#685921, @jroelofs wrote:

> In https://reviews.llvm.org/D30339#685919, @rmaprath wrote:
>
> > Perhaps change `config.h` and remove the definition there and adjust other places accordingly?
> >
> > The current form is very easy to trip over.
>
>
> Eric's point is that LIBCXXABI_BAREMETAL is a 0/1 flag, not a defined/not-defined flag. Please don't change from one form to the other... it's disruptive to build systems.


I actually think it's better to maintain consistency between libc++ and libc++abi. And libc++ never uses 0/1 flags. So I would rather see a fix in `config.h`.

Frankly I don't care that it is disruptive to build systems unless it's the build system owned by LLVM.


Repository:
  rL LLVM

https://reviews.llvm.org/D30339





More information about the cfe-commits mailing list