[PATCH] D111824: [llvm-profgen] Fix bug of populating profile symbol list

[Wenlei He via Phabricator via llvm-commits]

wenlei added inline comments.


================
Comment at: llvm/tools/llvm-profgen/ProfiledBinary.cpp:532
+        continue;
+      SymbolList.add(Name, true);
+    }
----------------
How do we deal with symbols without address range (`getAddressRanges`)? The old tool skips those, and this seems different. I'm not sure when we would have a function without address range though.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D111824