[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