[Openmp-commits] [PATCH] D118934: [OpenMP] Completely remove old device runtime

[Joseph Huber via Phabricator via Openmp-commits]

jhuber6 added inline comments.


================
Comment at: clang/include/clang/Driver/Options.td:2470
   Group<f_Group>, Flags<[CC1Option, NoArgumentUnused, HelpHidden]>;
-defm openmp_target_new_runtime: BoolFOption<"openmp-target-new-runtime",
-  LangOpts<"OpenMPTargetNewRuntime">, DefaultTrue,
----------------
tianshilei1992 wrote:
> Since LLVM 14 has this flags, we probably want to keep this argument, mark it as deprecated, and emit a warning saying there is no old runtime anymore.
What's LLVM's policy on deprecating flags? I would prefer to remove it but keeping it as a no-op would be fine.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D118934