[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