[PATCH] D36480: [LLVM] Change to Verifier to allow Fortran CHARACTER types in debug information

[Adrian Prantl via Phabricator via llvm-commits]

aprantl added a comment.

In https://reviews.llvm.org/D36480#841252, @schweitz wrote:

> The argument that the DWARF presentation to existing Fortran debug vendors must be changed doesn't seem too compelling.


I'm having a little difficulty to understand what you are saying, so let me attempt to rephrase this in my own words.
Am I understanding correctly that you want the DWARF representation of fixed-size Fortran strings to be:

  DW_TAG_string_type:
    DW_AT_name = "character(3)"
    DW_AT_byte_size = 3

?

As I said in my previous reply, I see two reasonable options for representing Fortran strings in *LLVM IR*, we could either using DIDerivedType and extending DICompositeType. In both cases, we can find a solution for dropping the base-`DW_AT_type` from `DW_TAG_string` when it isn't necessary. The decision for which of the two IR representations to use mostly depends on what plans you have for variable-length strings and arrays in your frontend.


https://reviews.llvm.org/D36480