[llvm] Add initial support for SPE brstack format (PR #129231)

Paschalis Mpeis via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 4 01:30:46 PDT 2025


================
@@ -998,9 +1008,21 @@ ErrorOr<LBREntry> DataAggregator::parseLBREntry() {
   if (std::error_code EC = MispredStrRes.getError())
     return EC;
   StringRef MispredStr = MispredStrRes.get();
-  if (MispredStr.size() != 1 ||
-      (MispredStr[0] != 'P' && MispredStr[0] != 'M' && MispredStr[0] != '-')) {
-    reportError("expected single char for mispred bit");
+  // SPE brstack mispredicted flags might be two characters long: 'PN' or 'MN'.
----------------
paschalis-mpeis wrote:

nit:
```suggestion
  // SPE brstack mispredicted flags might be up to two characters long: 'PN' or 'MN'.
```


Can add that 'N' is optional.


https://github.com/llvm/llvm-project/pull/129231


More information about the llvm-commits mailing list