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

[John Reagan via Phabricator via llvm-commits]

JohnReagan added a comment.

While I normally would recommend NOT encoding the language name in the constants (I'll be making that suggestion in "part 2" in just a bit), these symbols are clearly Fortran-only (you OR in bit DIFlagFortran 30).  The text in the DWARF4 standard also makes the same case but doesn't come out and say Fortran-only but just "These attributes are not relevant for languages that do not support similar keywords or syntax."  So should they be named DIFlagFortranPure, DIFlagFortranElemental, DIFlagFortranRecursive or is that too heavy handed?


Repository:
  rL LLVM

https://reviews.llvm.org/D54043