[clang] [llvm] [LinkerWrapper] Always pass `-flto` if the linker supports it (PR #102972)

Joel E. Denny via cfe-commits cfe-commits at lists.llvm.org
Tue Aug 13 09:00:32 PDT 2024


jdenny-ornl wrote:

> > > Updated to pass `-O2` instead, this was the actual behavior that passed, so it's mostly not a functional change.
> > 
> > 
> > So you just changed the tests not to use -O1? Doesn't this patch then represent a regression in what's supported?
> 
> I wouldn't consider something "supported" if it only worked because we overruled the user's `-O1` flag and replaced it with `-O2`.

Cases that used to compile successfully (even if more optimized than expected) will no longer.  Sounds like a regression.

https://github.com/llvm/llvm-project/pull/102972


More information about the cfe-commits mailing list