[all-commits] [llvm/llvm-project] c8c28a: [Driver] Place -mharden-sls= in m_Group

Fangrui Song via All-commits all-commits at lists.llvm.org
Sun Jun 11 08:59:56 PDT 2023


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: c8c28ac5caf875771f9a424e46255948004cbeab
      https://github.com/llvm/llvm-project/commit/c8c28ac5caf875771f9a424e46255948004cbeab
  Author: Fangrui Song <i at maskray.me>
  Date:   2023-06-11 (Sun, 11 Jun 2023)

  Changed paths:
    M clang/include/clang/Driver/Options.td
    M clang/test/Driver/x86-target-features.c

  Log Message:
  -----------
  [Driver] Place -mharden-sls= in m_Group

Fix https://github.com/llvm/llvm-project/issues/63237

With only a link action, we claim all CompileOnly_Group options (including -f*,
-m*, -i*, etc). -mharden-sls= is not in a Group and therefore not claimed,
leading to a spurious -Wunused-command-line-argument warning, and after
5548843d692a92a7840f14002debc3cebcb3cdc3, an error.




More information about the All-commits mailing list