[clang] Deprecate order file instrumentation (PR #121514)

[Ellis Hoag via cfe-commits]

================
@@ -8010,15 +8010,19 @@ void Clang::ConstructJob(Compilation &C, const JobAction &JA,
     }
   }
 
-  if (Args.hasArg(options::OPT_forder_file_instrumentation)) {
-     CmdArgs.push_back("-forder-file-instrumentation");
-     // Enable order file instrumentation when ThinLTO is not on. When ThinLTO is
-     // on, we need to pass these flags as linker flags and that will be handled
-     // outside of the compiler.
-     if (!IsUsingLTO) {
-       CmdArgs.push_back("-mllvm");
-       CmdArgs.push_back("-enable-order-file-instrumentation");
-     }
+  if (const Arg *A =
+          Args.getLastArg(options::OPT_forder_file_instrumentation)) {
+    D.Diag(diag::warn_drv_deprecated_arg)
+        << A->getAsString(Args) << /*hasReplacement=*/true
+        << "-mllvm -pgo-temporal-instrumentation";
----------------
ellishg wrote:

`-pgo-temporal-instrumentation` is stable and a significant improvement over `-forder-file-instrumentation`. I'm happy to help answer any questions about migrating to IRPGO, and I don't mind delaying the removal if necessary. 

I suppose I could add a frontend flag similar to how https://github.com/llvm/llvm-project/pull/109837 added [`-fprofile-generate-cold-function-coverage`](https://clang.llvm.org/docs/ClangCommandLineReference.html#cmdoption-clang-fprofile-generate-cold-function-coverage). I'll think about the naming.

https://github.com/llvm/llvm-project/pull/121514