[all-commits] [llvm/llvm-project] 7d3886: [ELF] Rename --[no-]lto-new-pass-manager to --[no-...

Fangrui Song via All-commits all-commits at lists.llvm.org
Wed Dec 9 17:53:54 PST 2020


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 7d38861ce300a8e19eacbf92e63ea7a4c43ec1c9
      https://github.com/llvm/llvm-project/commit/7d38861ce300a8e19eacbf92e63ea7a4c43ec1c9
  Author: Fangrui Song <i at maskray.me>
  Date:   2020-12-09 (Wed, 09 Dec 2020)

  Changed paths:
    M lld/ELF/Driver.cpp
    M lld/ELF/Options.td
    M lld/test/ELF/lto/new-pass-manager.ll
    M lld/test/ELF/lto/verify-invalid.ll

  Log Message:
  -----------
  [ELF] Rename --[no-]lto-new-pass-manager to --[no-]lto-legacy-pass-manager

Normally we should not delete options. However, the Clang driver passes
`-plugin-opt={new,legacy}-pass-manager` instead of
`--[no-]lto-legacy-pass-manager` (`-plugin-opt=new-pass-manager` has been used
since 7.0), and it is unlikely anyone will use the `--lto-*` style options directly.

So let's rename them to be consistent with the Clang driver option names.

Reviewed By: aeubanks

Differential Revision: https://reviews.llvm.org/D92988




More information about the All-commits mailing list