[PATCH] D95929: [CSSPGO][llvm-profgen] Add brackets for context id to support extended binary format

Lei Wang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 5 15:58:07 PST 2021


wlei added inline comments.


================
Comment at: llvm/test/tools/llvm-profgen/recursion-compression-pseudoprobe.test:10
 ; CHECK-UNCOMPRESS:  3: 1
-; CHECK-UNCOMPRESS:  4: 1
-; CHECK-UNCOMPRESS:  7: 1 fb:1
+; CHECK-UNCOMPRESS:  5: 1
+; CHECK-UNCOMPRESS:  8: 1 fa:1
----------------
hoy wrote:
> What causes this diff? Looks like the output order changed.
Exactly, because now it ordered in the context with bracket, then the ascii code of ']' is greater than other alphabet or ':', the following test changes are the same.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D95929



More information about the llvm-commits mailing list