[PATCH] D123874: [Clang][IA] support -generate-unused-section-symbols={yes|no}

2022-04-22 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay requested changes to this revision. MaskRay added a comment. This revision now requires changes to proceed. In D123874#3454723 , @MaskRay wrote: >> Add support to clang (-Wa,-generate-unused-section-symbols={yes|no}) and >> llvm-mc. > > > >>

[PATCH] D123874: [Clang][IA] support -generate-unused-section-symbols={yes|no}

2022-04-15 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay added a comment. > Add support to clang (-Wa,-generate-unused-section-symbols={yes|no}) and > llvm-mc. > This feature might be used by the Linux kernel. > https://lore.kernel.org/linux-toolchains/ylluqpk4cwzeh...@hirez.programming.kicks-ass.net/ > This has been supported by GNU

[PATCH] D123874: [Clang][IA] support -generate-unused-section-symbols={yes|no}

2022-04-15 Thread Fangrui Song via Phabricator via cfe-commits
MaskRay added a comment. I wish we don't do this. https://sourceware.org/pipermail/binutils/2022-March/119940.html (`RFC: GAS: Add option to generate unused section symbols`): > This problem was fixed in the kernel (and backported by distros to their kernels if binutils was updated); it's

[PATCH] D123874: [Clang][IA] support -generate-unused-section-symbols={yes|no}

2022-04-15 Thread Nick Desaulniers via Phabricator via cfe-commits
nickdesaulniers created this revision. nickdesaulniers added a reviewer: MaskRay. Herald added subscribers: StephenFan, dexonsmith, hiraditya. Herald added a project: All. nickdesaulniers requested review of this revision. Herald added projects: clang, LLVM. Herald added subscribers: llvm-commits,