[PATCH] D122584: [DebugInfo] Use DW_ATE_signed encoding when creating a Fortran array index type.
Adrian Prantl via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Apr 5 15:44:22 PDT 2022
aprantl added a comment.
Interesting. I wasn't aware of the prior art in `DwarfUnit::getLowerBound()`. What do you think about adding a `getArrayIndexEncoding(language)` function to `llvm/BinaryFormat/Dwarf.h`? This would group the per-language defaults in the DWARF library instead of the backend. Otherwise we could land this patch as is, too.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D122584/new/
https://reviews.llvm.org/D122584
More information about the llvm-commits
mailing list