[PATCH] D109637: [llvm-profgen] Ignore broken LBR samples

Wenlei He via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 14 09:49:48 PDT 2021


wenlei added inline comments.


================
Comment at: llvm/tools/llvm-profgen/PerfReader.cpp:428
+    if (Addresses[0].substr(2).getAsInteger(16, Src) ||
+        Addresses[1].substr(2).getAsInteger(16, Dst))
+      break;
----------------
Print a warning so we know when this happens? Ideally the warning should also include the invalid Line `TraceIt.getCurrentLine()`, so we know what is the cause. 


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109637



More information about the llvm-commits mailing list