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