[PATCH] D134822: [PERF2BOLT] Do not issue parsing error on weird build ids

Rafael Auler via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Sep 28 14:42:21 PDT 2022


This revision was landed with ongoing or failed builds.
This revision was automatically updated to reflect the committed changes.
Closed by commit rGe549ac072b55: [PERF2BOLT] Do not issue parsing error on weird build ids (authored by rafauler).

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D134822

Files:
  bolt/lib/Profile/DataAggregator.cpp


Index: bolt/lib/Profile/DataAggregator.cpp
===================================================================
--- bolt/lib/Profile/DataAggregator.cpp
+++ bolt/lib/Profile/DataAggregator.cpp
@@ -2147,7 +2147,8 @@
 
   // If one of the strings is missing, don't issue a parsing error, but still
   // do not return a value.
-  if (ParsingBuf[0] == '\n')
+  consumeAllRemainingFS();
+  if (checkAndConsumeNewLine())
     return NoneType();
 
   ErrorOr<StringRef> NameStr = parseString(FieldSeparator, true);


-------------- next part --------------
A non-text attachment was scrubbed...
Name: D134822.463676.patch
Type: text/x-patch
Size: 507 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20220928/8c1096be/attachment.bin>


More information about the llvm-commits mailing list