[PATCH] D21489: [codeview] Added support for bitfield type

Amjad Aboud via llvm-commits llvm-commits at lists.llvm.org
Mon Jun 27 12:06:40 PDT 2016


aaboud added a comment.

> I don't think this is a good idea. It would be far cheaper to encode the offset of the underlying allocation in the extraData.


Please, review the last version of the patch and the patch for clang (http://reviews.llvm.org/D21766).
I think these patches complete the support for all bitfield cases.
However, if you have a better way to implement this support, I do not mind to abandon these patches.

Thanks.


http://reviews.llvm.org/D21489





More information about the llvm-commits mailing list