[PATCH] D51990: [DebugInfo] Fix emitting of bit offset for ObjC

Jonas Devlieghere via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Wed Sep 12 09:04:52 PDT 2018


JDevlieghere added inline comments.


================
Comment at: clang/lib/CodeGen/CGDebugInfo.cpp:2369
+              ? CGM.getObjCRuntime().ComputeBitfieldBitOffset(CGM, ID, Field)
+              : 0;
     } else {
----------------
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! 


Repository:
  rC Clang

https://reviews.llvm.org/D51990





More information about the cfe-commits mailing list