[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