[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