[llvm] [DWARF] Use ULEB128 and not just one byte for directory indices (PR #109067)

Jonas Devlieghere via llvm-commits llvm-commits at lists.llvm.org
Tue Sep 24 09:48:30 PDT 2024


Daniel =?utf-8?q?Rodríguez_Troitiño?=,
Daniel =?utf-8?q?Rodríguez_Troitiño?=,
Daniel =?utf-8?q?Rodríguez_Troitiño?Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/109067 at github.com>


https://github.com/JDevlieghere approved this pull request.

Given that we know the number of indices, we could always emit the most efficient encoding. That said, we're emitting this only once per line table, so I'm not particularly worried about the additional byte and simplifies the implementation. LGTM.

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


More information about the llvm-commits mailing list