[PATCH] D36480: [LLVM] Change to Verifier to allow Fortran CHARACTER types in debug information
Adrian Prantl via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Tue Aug 8 16:40:39 PDT 2017
aprantl accepted this revision.
aprantl added a comment.
This revision is now accepted and ready to land.
I guess having this test in test/DebugInfo makes more sense than a "positive" test in test/Verifier.
Somewhat related question: Do you also need/want to add a DIBuilder API to create the type?
================
Comment at: string-type.ll :16
+!2 = !{ i32 1, !"Debug Info Version", i32 3 }
+!3 = !DIFile(filename: "fortran", directory: "/")
+!5 = !{ }
----------------
shouldn't this be something.f90? :-)
https://reviews.llvm.org/D36480
More information about the llvm-commits
mailing list