JDevlieghere added inline comments.
================
Comment at: clang/lib/CodeGen/CGDebugInfo.cpp:2369
+ ? CGM.getObjCRuntime().ComputeBitfieldBitOffset(CGM, ID, Field)
+ : 0;
} else {
----------------
JDevlieghere wrote:
> aprantl wrote:
> > aprantl wrote:
> > > It might help to attempt some git blame archeology.
> > > Judging from the comment, it sounds like the debugger is supposed to
> > > query the runtime for the *byte* offset and then add the bit offset from
> > > DWARF? Could that make sense?
> > If that is the case, we'd need to relax llvm-dwarfdump --verify to accept
> > this and make sure LLDB does the right thing instead.
> Ah I see, yeah that sounds reasonable and explains the comment which I
> interpreted differently. Thanks!
btw it was added in rL167503.
Repository:
rC Clang
https://reviews.llvm.org/D51990
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
