[llvm] [NFC][InstrProf]Refactor readPGOFuncNameStrings (PR #71566)
Mingming Liu via llvm-commits
llvm-commits at lists.llvm.org
Tue Nov 7 11:34:26 PST 2023
================
@@ -469,14 +464,14 @@ class InstrProfSymtab {
/// until before it is used. See also \c create(StringRef) method.
Error create(object::SectionRef &Section);
- /// This interface is used by reader of CoverageMapping test
- /// format.
- inline Error create(StringRef D, uint64_t BaseAddr);
-
/// \c NameStrings is a string composed of one of more sub-strings
/// encoded in the format described in \c collectPGOFuncNameStrings.
/// This method is a wrapper to \c readPGOFuncNameStrings method.
- inline Error create(StringRef NameStrings);
+ Error create(StringRef NameStrings);
+
+ /// This interface is used by reader of CoverageMapping test
+ /// format.
+ inline Error create(StringRef D, uint64_t BaseAddr);
----------------
minglotus-6 wrote:
I think this is an artifact of 'diff' UI.
I swapped the order of two functions(https://github.com/llvm/llvm-project/blob/fb8ff4cdaecb7fe498d4e3a9cbcb33d40cccd29a/llvm/include/llvm/ProfileData/InstrProf.h#L472-L479) primarily because, the second `create` no longer has an 'inline' keyword (since its definition is moved to cpp files).
https://github.com/llvm/llvm-project/pull/71566
More information about the llvm-commits
mailing list