search for: emitcoffstaticoffset

...just by using the MCAsmStreamer. > > Do you think we should really write a dumper too? > > That's kinda hard and we don't plan to fully support the CodeView format > yet... > > I tried my patch on Chromium and it hit the llvm_unreachable I wrote > in WinCOFFStreamer::EmitCOFFStaticOffset(). > Now that it also supports using a fixup to calculate the offset (that > happens as the second pass, not supported by MCAsmStreamer, right?), I > think I do have to write at least a simple dumper... > > Any hints on how to do that? Should it be a separate app or built into > s...

...gt; tests without a dumper - just by using the MCAsmStreamer. > Do you think we should really write a dumper too? > That's kinda hard and we don't plan to fully support the CodeView format yet... I tried my patch on Chromium and it hit the llvm_unreachable I wrote in WinCOFFStreamer::EmitCOFFStaticOffset(). Now that it also supports using a fixup to calculate the offset (that happens as the second pass, not supported by MCAsmStreamer, right?), I think I do have to write at least a simple dumper... Any hints on how to do that? Should it be a separate app or built into some COFF reader readily avail...

...er. >> > Do you think we should really write a dumper too? >> > That's kinda hard and we don't plan to fully support the CodeView format >> > yet... >> >> I tried my patch on Chromium and it hit the llvm_unreachable I wrote >> in WinCOFFStreamer::EmitCOFFStaticOffset(). >> Now that it also supports using a fixup to calculate the offset (that >> happens as the second pass, not supported by MCAsmStreamer, right?), I >> think I do have to write at least a simple dumper... >> >> Any hints on how to do that? Should it be a separate app...

Attached is a slightly updated patch. (it doesn't include D2222 yet). 2013/11/19 Eric Christopher <echristo at gmail.com>: > In general I do think we're going to need to abstract it out as much > as possible. I'm not sure what the previous patch looks like, but > abstracting the interface out would be general goodness for this. We > can talk about designs for that as

...in the code where I can find the clues for that? >> I'm not yet familiar with this part of the project... >> lib/CodeGen/AsmPrinter/DwarfDebug.cpp has some checks for triple/os. > > I tried my patch on Chromium and it hit the llvm_unreachable I wrote > in WinCOFFStreamer::EmitCOFFStaticOffset(). > Now that it also supports using a fixup to calculate the offset (that > happens as the second pass, not supported by MCAsmStreamer, right?), I > think I do have to write at least a simple dumper... > > Any hints on how to do that? Should it be a separate app or built into > s...

...amiliar with this part of the project... > >> > > lib/CodeGen/AsmPrinter/DwarfDebug.cpp has some checks for triple/os. OK, I use the Triple().isOSWindows() now. > > > > I tried my patch on Chromium and it hit the llvm_unreachable I wrote > > in WinCOFFStreamer::EmitCOFFStaticOffset(). > > Now that it also supports using a fixup to calculate the offset (that > > happens as the second pass, not supported by MCAsmStreamer, right?), I > > think I do have to write at least a simple dumper... > > > > Any hints on how to do that? Should it be a separate...