[PATCH] D129594: [InstrProf] Add options to profile function groups

[Ellis Hoag via Phabricator via cfe-commits]

ellis added inline comments.


================
Comment at: clang/lib/CodeGen/CodeGenModule.cpp:2871
+  if (NumGroups > 1) {
+    auto Group = llvm::MD5Hash(Fn->getName()) % NumGroups;
+    if (Group != getCodeGenOpts().ProfileSelectedFunctionGroup)
----------------
ianlevesque wrote:
> In D87953 I used crc32 to avoid computing an md5 hash on every function name.
I see, I'll change that. Thanks!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D129594