[PATCH] D54114: Adding debug info to support Fortran (part 2)

[Paul Robinson via Phabricator via llvm-commits]

probinson added a comment.

Sorry for being late to the party...  I'm probably repeating some other comments but wanted to get these thoughts down.
Existing DI should handle an array with compile-time-constant bounds, so would not need anything new.
Variable-bound arrays are available in a variety of languages, not excepting C (VLAs).  Again I'd have thought that existing DI would be able to handle this case, although I'm less sure of that.
DWARF did add a bunch of stuff for the newer Fortran array features, I'll have to re-acquaint myself with those before commenting.


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D54114/new/

https://reviews.llvm.org/D54114