[PATCH] D47953: [builtin] Add bitfield support for __builtin_dump_struct

[Paul Semel via Phabricator via cfe-commits]

paulsemel added a comment.

In https://reviews.llvm.org/D47953#1143044, @dblaikie wrote:

> This doesn't seem to build for me - so hard to debug/probe it:
>
> llvm/src/tools/clang/lib/CodeGen/CGBuiltin.cpp:1264:65: error: no viable conversion from 'clang::QualType' to 'llvm::Type *'
>
>   CGF.CGM.getDataLayout().getTypeSizeInBits(CanonicalType),
>                                             ^~~~~~~~~~~~~
>
> llvm/src/include/llvm/IR/DataLayout.h:560:53: note: passing argument to parameter 'Ty' here
>  inline uint64_t DataLayout::getTypeSizeInBits(Type *Ty) const {
>
>   ^
>
> 1 error generated.


Very sorry about it, I should have paid more attention..


Repository:
  rC Clang

https://reviews.llvm.org/D47953