h-vetinari wrote:
I'm trying another approach here (now that #96171 showed we cannot get rid of
`llvm_check_compiler_linker_flag` anyway) -- instead of doing the manual
work-arounds for the linker flags & static targets everywhere, teach the
existing function to do it (optionally).
There's not so many callers in the code base that this looks infeasible per
se, but people might want to design the API differently (e.g. have a dedicated
helper function for the case under consideration here). I'm also expecting this
to break in various ways until I've figured out the right invocations
everywhere. There's also several implementations of
`{target_,}add_flag_if_supported`, which _might_ have to expose similar
arguments. Or we unify all those functions as well. Happy to iterate, or have
someone else take over - I'll be the first to admit that CMake is not my forte
😅
https://github.com/llvm/llvm-project/pull/93429
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
