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

[Duncan P. N. Exon Smith via llvm-commits]

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
> 
> <D21614.61581.patch>

> On 2016-Jun-30, at 19:48, Reid Kleckner <rnk at google.com> wrote:
> 
> This revision was automatically updated to reflect the committed changes.
> Closed by commit rL274325: [codeview] Add DISubprogram::ThisAdjustment (authored by rnk).
> 
> Changed prior to commit:
>  http://reviews.llvm.org/D21614?vs=61702&id=62457#toc
> 
> Repository:
>  rL LLVM
> 
> http://reviews.llvm.org/D21614
> 
> Files:
>  llvm/trunk/include/llvm/IR/DIBuilder.h
>  llvm/trunk/include/llvm/IR/DebugInfoMetadata.h
>  llvm/trunk/lib/AsmParser/LLParser.cpp
>  llvm/trunk/lib/Bitcode/Reader/BitcodeReader.cpp
>  llvm/trunk/lib/Bitcode/Writer/BitcodeWriter.cpp
>  llvm/trunk/lib/CodeGen/AsmPrinter/CodeViewDebug.cpp
>  llvm/trunk/lib/IR/AsmWriter.cpp
>  llvm/trunk/lib/IR/DIBuilder.cpp
>  llvm/trunk/lib/IR/DebugInfoMetadata.cpp
>  llvm/trunk/lib/IR/LLVMContextImpl.h
>  llvm/trunk/test/Assembler/disubprogram.ll
>  llvm/trunk/test/Bitcode/upgrade-subprogram-this.ll
>  llvm/trunk/test/Bitcode/upgrade-subprogram-this.ll.bc
>  llvm/trunk/test/DebugInfo/COFF/virtual-methods.ll
>  llvm/trunk/unittests/IR/MetadataTest.cpp
> 
> <D21614.62457.patch>