mesos-dev mailing list archives

Site index · List index
Message view « Date » · « Thread »
Top « Date » · « Thread »
From Joris Van Remoortere <jo...@mesosphere.io>
Subject Re: Building on OS X 10.12
Date Mon, 12 Dec 2016 19:17:14 GMT
There are a significant number of developer and runtime performance
benefits from turning this flag on.
In my opinion it is also a dangerous flag to turn on by default without a
strict set of rules for our codebase to ensure that we don't accidentally:

   - have multiple instances of static variables when we think they are a
   singleton
   - run into inequality when we expect equality for comparison of in-lined
   function pointers (For example when building a vtable in a library for
   something like variant / visitor)

Although the likelihood that our codebase would suffer from these is low, I
would prefer to have clang-tidy support and have some rule checkers to
ensure that we can turn this flag on by default and know we will catch any
future code that may break these rules.

@James have you done any validation of the codebase and the libraries we
depend on to ensure this is safe?

Joris

—
*Joris Van Remoortere*
Mesosphere

On Mon, Dec 5, 2016 at 1:16 PM, James Peach <jorgar@gmail.com> wrote:

>
> > On Dec 2, 2016, at 10:54 PM, Jie Yu <yujie.jay@gmail.com> wrote:
> >
> > Another tip. If you are on macOS sierra, you might notice the linking is
> > extremely slow using the default clang.
> >
> > Using CXXFLAGS `-fvisibility-inlines-hidden` will greatly speedup the
> > linking.
>
> Is there a reason we should not always do this? It reduces the number of
> exported symbols in libmesos.so from 250K to 100K.
>
> J

Mime
  • Unnamed multipart/alternative (inline, None, 0 bytes)
View raw message