[llvm] [Support] Add decodeULEB128AndInc/decodeSLEB128AndInc (PR #85739)
Fangrui Song via llvm-commits
llvm-commits at lists.llvm.org
Thu Mar 21 19:22:06 PDT 2024
MaskRay wrote:
The tablegen use case guarantees that there is no error. The `Shift>63` check is pure overhead.
Use cases that require rigid error checking can probably use `llvm/include/llvm/Support/DataExtractor.h` `getULEB128`, but the overhead would be substantial.
At any rate, the error parameters could be added to `*AndInc` but I have found a good use case yet.
For efficient ULEB128 for the primary use cases (debug info, etc), perhaps we should unroll the first iteration.
https://github.com/llvm/llvm-project/pull/85739
More information about the llvm-commits
mailing list