[PATCH] D86305: [DebugInfo]Added support for representing Fortran assumed length strings

Sourabh Singh Tomar via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Aug 20 11:09:20 PDT 2020


SouraVX added inline comments.


================
Comment at: llvm/include/llvm/IR/DebugInfoMetadata.h:852
+                    (Tag, Name, StringLength, StringLengthExp, SizeInBits,
+                     AlignInBits, Encoding))
+
----------------
aprantl wrote:
> Are these copy&paste errors?
Nope, just for backward compatilbility I guess ? just like other nodes. Notice one expects `MDString` and other one `StringRef`


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D86305/new/

https://reviews.llvm.org/D86305



More information about the llvm-commits mailing list