[PATCH] D122584: [DebugInfo] Use DW_ATE_signed encoding when creating a Fortran array index type.

David Blaikie via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Mar 28 15:45:15 PDT 2022


dblaikie added a comment.

In D122584#3411769 <https://reviews.llvm.org/D122584#3411769>, @aprantl wrote:

> I'm not sure this would work in LTO when a Fortran function is inlined into a C function. I think it would be more general to add an optional base type (or encoding) to DISubrange.

I /think/ this would still work - types' scope chains would lead back to the fortran CU as their root & so that'd be correct? Worth an experiment/validation though (but may not need a committed test case covering this).


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