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

Adrian Prantl via llvm-commits llvm-commits at lists.llvm.org
Wed Jun 22 15:27:20 PDT 2016


aprantl added a comment.

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?

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


http://reviews.llvm.org/D21614





More information about the llvm-commits mailing list