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

[Reid Kleckner via llvm-commits]

rnk added a comment.

In http://reviews.llvm.org/D21614#465134, @aprantl wrote:

> Is this something that maybe is only needed on function definitions versus declarations?
>  We're planning to split these up anyway, maybe we can make this field optional after/during the split?


Unfortunately, it would need to live on the declaration, because we need it when emitting the class member list.

> I think this also needs a binary bitcode test that tests autoupgrading from the old format.


Sure.


http://reviews.llvm.org/D21614