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

Arthur Eubanks via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Apr 11 14:32:37 PDT 2022


aeubanks updated this revision to Diff 422038.
aeubanks added a comment.

remove help text instead of modifying generated docs


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.422038.patch
Type: text/x-patch
Size: 1731 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20220411/98b46309/attachment.bin>


More information about the cfe-commits mailing list