[llvm] [llvm] simplify and clean-up DemangleConfig.h (PR #149163)
Jordan Rupprecht via llvm-commits
llvm-commits at lists.llvm.org
Tue Jul 22 18:56:34 PDT 2025
rupprecht wrote:
> It should be safe for the Demangle component library to depend on Support, so there is no need for it to maintain copies of macros defined in llvm/Support/Compiler.h.
This commit creates a library layering violation because Support already depends on Demangle, e.g. https://github.com/llvm/llvm-project/blob/324773e238026c5d4f501213678a89bf411e1509/llvm/lib/Support/Unix/Signals.inc#L38
Is there another way you can land this commit w/o creating a cycle?
https://github.com/llvm/llvm-project/pull/149163
More information about the llvm-commits
mailing list