[NFC] [sanitizer] fix inaccurate comment

[fmayer]

No description provided.

[llvmbot]

@llvm/pr-subscribers-clang

Author: Florian Mayer (fmayer)

Changes

---

Full diff: https://github.com/llvm/llvm-project/pull/145809.diff

1 Files Affected:

• (modified) clang/include/clang/Basic/Sanitizers.h (+2-2)

diff --git a/clang/include/clang/Basic/Sanitizers.h b/clang/include/clang/Basic/Sanitizers.h
index 3782deb5962bc..b299de99157e7 100644
--- a/clang/include/clang/Basic/Sanitizers.h
+++ b/clang/include/clang/Basic/Sanitizers.h
@@ -218,8 +218,8 @@ bool parseSanitizerWeightedValue(StringRef Value, bool AllowGroups,
 void serializeSanitizerSet(SanitizerSet Set,
                            SmallVectorImpl<StringRef> &Values);
 
-/// Serialize a SanitizerMaskCutoffs into values for -fsanitize= or
-/// -fno-sanitize=.
+/// Serialize a SanitizerMaskCutoffs into values for
+// -fsanitize-skip-hot-cutoff=.
 void serializeSanitizerMaskCutoffs(const SanitizerMaskCutoffs &Cutoffs,
                                    SmallVectorImpl<std::string> &Values);
 

[thurstond]

There is nothing about this function that is specific to -fsanitize-skip-hot-cutoffs. It's general enough that it could be used for any future -fsanitize- flags that need cutoffs.

[fmayer]

There is nothing about this function that is specific to -fsanitize-skip-hot-cutoffs. It's general enough that it could be used for any future -fsanitize- flags that need cutoffs.

So, just remove the "... for fsanitize"?

[thurstond]

There is nothing about this function that is specific to -fsanitize-skip-hot-cutoffs. It's general enough that it could be used for any future -fsanitize- flags that need cutoffs.

So, just remove the "... for fsanitize"?

Yes. Drop the " for -fsanitize= or /// -fno-sanitize=". Just "Serialize a SanitizerMaskCutoffs into values." It's cleaner.