[clang] [lld] [llvm] [llvm][lld][clang] Delay initializing TargetOptions in LTO builds until a Triple is available (PR #179509)

[Paul Kirth via llvm-commits]

================
@@ -140,14 +141,17 @@ std::string llvm::computeLTOCacheKey(
     Hasher.update(ArrayRef<uint8_t>(&I, 1));
   };
   AddString(Conf.CPU);
+  TargetOptions Opts = codegen::InitTargetOptionsFromCodeGenFlags(TT);
+  Conf.ModifyTargetOptions(Opts);
+
   // FIXME: Hash more of Options. For now all clients initialize Options from
----------------
ilovepi wrote:

hmm, maybe worth asking on discourse if this is used/important downstream ... If not, I'd propose we drop it in the next release, or at least deprecate it in preparation for removal.

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