[PATCH] D54043: Adding debug info to support Fortran (part 1)

[Paul Robinson via Phabricator via llvm-commits]

probinson added a comment.

Even when DWARF adds something in response to the needs of a specific language, it doesn't use language names in the tags or attributes. So, I think putting "Fortran" in the symbol names here would be inconsistent with that.  It's fine to describe it as a Fortran attribute in commentary.
As a hypothetical example, C++ has the concept of a "pure" virtual method declaration, and somebody might come up with a reason to mark those with the new Pure flag.
Which is another reason not to start overloading the existing flags based on assumptions about which languages use which flags. I've started sketching out what it would take to add a separate DISPFlags type/field in DISubprogram.


Repository:
  rL LLVM

https://reviews.llvm.org/D54043