[PATCH] D82881: [DEBUGINFO]Fix debug info for packed bitfields.

Wed Jul 8 10:38:45 PDT 2020

ABataev added a comment.

In D82881#2139589 <https://reviews.llvm.org/D82881#2139589>, @tra wrote:

> In D82881#2138942 <https://reviews.llvm.org/D82881#2138942>, @ABataev wrote:
>
> > The bug I'm trying to fix is the incompatibility with NVPTX ptxas compiler. It does not allow signed integers in debug sections. Would it be good to emit bit_offset as `DW_FORM_udata` for NVPTX target to fix incompatibility? Checked that it works with ptxas.
>
>
> Looks like we'll need to teach MCAsmStreamer to handle 'unsigned-only' data directives. Right now it always prints a signed value. Making dwarf use unsigned values is just one of the ways to trigger the issue.
>
> I'll get it fixed.

I have a patch for it already. It is quite simple, just need to set the Form to `DW_FORM_udata` and everything work. I can update this patch, if you want to try the fix.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D82881/new/

https://reviews.llvm.org/D82881