[PATCH] D123547: [docs] Mention that we are in the process of removing the legacy PM for the optimization pipeline

Arthur Eubanks via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 12 10:48:27 PDT 2022


This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rG9faab435a357: [docs] Mention that we are in the process of removing the legacy PM for theā€¦ (authored by aeubanks).

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D123547/new/

https://reviews.llvm.org/D123547

Files:
  clang/include/clang/Driver/Options.td
  llvm/docs/NewPassManager.rst


Index: llvm/docs/NewPassManager.rst
===================================================================
--- llvm/docs/NewPassManager.rst
+++ llvm/docs/NewPassManager.rst
@@ -480,12 +480,9 @@
 the new PM, whereas the backend target-dependent code generation only works
 with the legacy PM.
 
-For the optimization pipeline, the new PM is the default PM. The legacy PM is
-available for the optimization pipeline by setting various compiler/linker
-flags, e.g. ``-flegacy-pass-manager`` for ``clang``.
-
-There will be efforts to deprecate and remove the legacy PM for the
-optimization pipeline in the future.
+For the optimization pipeline, the new PM is the default PM. Using the legacy PM
+for the optimization pipeline is deprecated and there are ongoing efforts to
+remove its usage.
 
 Some IR passes are considered part of the backend codegen pipeline even if
 they are LLVM IR passes (whereas all MIR passes are codegen passes). This
Index: clang/include/clang/Driver/Options.td
===================================================================
--- clang/include/clang/Driver/Options.td
+++ clang/include/clang/Driver/Options.td
@@ -1937,8 +1937,8 @@
   Alias<fglobal_isel>;
 defm legacy_pass_manager : BoolOption<"f", "legacy-pass-manager",
   CodeGenOpts<"LegacyPassManager">, DefaultFalse,
-  PosFlag<SetTrue, [], "Use the legacy pass manager in LLVM (deprecated, to be removed in a future release)">,
-  NegFlag<SetFalse, [], "Use the new pass manager in LLVM">,
+  PosFlag<SetTrue>,
+  NegFlag<SetFalse>,
   BothFlags<[CC1Option]>>, Group<f_clang_Group>;
 def fexperimental_new_pass_manager : Flag<["-"], "fexperimental-new-pass-manager">,
   Group<f_clang_Group>, Flags<[CC1Option]>, Alias<fno_legacy_pass_manager>;


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D123547.422293.patch
Type: text/x-patch
Size: 1731 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220412/3844c3a2/attachment.bin>


More information about the llvm-commits mailing list