[PATCH] D21614: [codeview] Add DISubprogram::ThisAdjustment

Duncan P. N. Exon Smith via llvm-commits llvm-commits at lists.llvm.org
Thu Jun 30 20:07:23 PDT 2016


dexonsmith added a subscriber: dexonsmith.
dexonsmith added a comment.

You never answered my question from a few weeks ago:

> Every 8 bytes counts for a lot since we have so many of these in LTO.  If we can find some way to make it optional (via subclassing?) that would be great.




> http://reviews.llvm.org/D21614

> 

> Files:

>  include/llvm/IR/DIBuilder.h

>  include/llvm/IR/DebugInfoMetadata.h

>  lib/AsmParser/LLParser.cpp

>  lib/Bitcode/Reader/BitcodeReader.cpp

>  lib/Bitcode/Writer/BitcodeWriter.cpp

>  lib/CodeGen/AsmPrinter/CodeViewDebug.cpp

>  lib/IR/AsmWriter.cpp

>  lib/IR/DIBuilder.cpp

>  lib/IR/DebugInfoMetadata.cpp

>  lib/IR/LLVMContextImpl.h

>  test/DebugInfo/COFF/virtual-methods.ll

>  unittests/IR/MetadataTest.cpp

> 

> <http://reviews.llvm.org/D21614.61581.patch>



Repository:
  rL LLVM

http://reviews.llvm.org/D21614





More information about the llvm-commits mailing list