[PATCH] D54043: Adding debug info to support Fortran (part 1)
John Reagan via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Mon Nov 5 12:04:22 PST 2018
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
More information about the llvm-commits
mailing list