================
@@ -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
----------------
teresajohnson wrote:
Does this comment need an update (at least s/Options/TargetOptions/), and is it
still valid? I.e. since this change initializes more of the TargetOptions from
the Triple, are there more things that potentially need to be hashed. Or should
the Triple itself be hashed?
https://github.com/llvm/llvm-project/pull/179509
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
