ldionne wrote:

> > > I am on the fence whether a driver option is really needed. It is a very 
> > > shallow layer of extra abstraction that a curious reader has to look 
> > > through. I guess I'll not object to this, if people really want to add it.
> > 
> > 
> > Setting macros that are reserved names is really unappealing for people. We 
> > want to provide a well integrated experience for library hardening and that 
> > includes providing a friendly option at the compiler level. Other sanity 
> > checking in the driver like erroring out when libc++ isn't in use is also 
> > good,
> 
> That pre-empts making it work for libstdc++ as suggested above...

Until we do something sane on libstdc++, I would error out in the Driver. As 
soon as we know what to do on libstdc++, we should do that instead and stop 
erroring out. Does that make sense? For example, in a next step we should 
discuss whether it makes sense for `-fstdlib-hardening={extensive,debug}` to 
turn on `_GLIBCXX_ASSERTIONS` on GCC. Maybe we even want that in `fast` mode? 
We can do anything, but we should make sure to keep the libstdc++ folks in the 
loop to make sure it makes sense.

https://github.com/llvm/llvm-project/pull/78763
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to