It looks like we might be able to test ${CMAKE_CXX_LINK_EXECUTABLE} to
exclude those that don't need --start-group/--end-group linker flags.However, this is getting pretty far past the original contributor patch. I'm more inclined to roll it back at this point until the whole thing gets more testing. On Tue, Jun 17, 2014 at 12:25 PM, Todd Fiala <[email protected]> wrote: > Ok - so given it's a linker related flags issue, what is the correct cmake > way to test out the linker being used? > > > On Tue, Jun 17, 2014 at 11:49 AM, Ed Maste <[email protected]> wrote: > >> On 17 June 2014 14:33, Todd Fiala <[email protected]> wrote: >> > >> > Hey Ed - can you test this out on FreeBSD? This is a patch that fixes >> cmake on MacOSX. I verified it works on MacOSX and Linux with cmake/ninja. >> > >> > The original code looks like it was opting in for the >> --start-group/--end-group linker options on FreeBSD, which this patch would >> change but might be a no-op on your end. >> >> It's odd that this is a compiler-dependent test, I'd expect it to be a >> linker issue. In any case, we definitely needed --start-group and >> --end-group in the past. If this test works with Linux though it will >> probably work with FreeBSD too. I'll test it. >> > > > > -- > Todd Fiala | Software Engineer | [email protected] | 650-943-3180 > -- Todd Fiala | Software Engineer | [email protected] | 650-943-3180
_______________________________________________ lldb-commits mailing list [email protected] http://lists.cs.uiuc.edu/mailman/listinfo/lldb-commits
