[PATCH] D117141: [CSSPGO] Print "csnested" instead of "preilnined" for SecFuncMetadata.

[Hongtao Yu via Phabricator via llvm-commits]

hoy added inline comments.


================
Comment at: llvm/lib/ProfileData/SampleProfReader.cpp:1292
     if (hasSecFlag(Entry, SecFuncMetadataFlags::SecFlagIsCSNested))
-      Flags.append("preinlined,");
+      Flags.append("csnested,");
     break;
----------------
wenlei wrote:
> nit: `csnested` -> `context-nested` because we use `context` for `SecFlagFullContext`
> 
> Additionally, would it make sense for `SecFlagIsCSNested` to be a flag for `SecProfSummaryFlags` just like `SecFlagFullContext`?
Yes, makes sense to have the flag for profile summary.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D117141/new/

https://reviews.llvm.org/D117141