search for: codeviewerror

...by MC don't depend on Object)? Any suggestions? Sink it into > BinaryFormat or name a new library? > > (it's quite a few headers by the looks of it: CodeView.h, Line.h, > SymbolRecord.h, CodeViewTypes.def, CodeViewSymbols.def, > CodeViewRegisters.def, TypeIndex.h, CVRecord.h, CodeViewError.h, > RecordSerialization.h - maybe some of those aren't needed, if they're > excessive includes or the like (they tend to seep in all over the place)) > > >> >> >>> include/llvm/Object/COFF.h includes >>>> include/llvm/DebugInfo/CodeView/CVDebugR...

...ssuming the bits needed by MC don't depend on Object)? Any suggestions? Sink it into BinaryFormat or name a new library? (it's quite a few headers by the looks of it: CodeView.h, Line.h, SymbolRecord.h, CodeViewTypes.def, CodeViewSymbols.def, CodeViewRegisters.def, TypeIndex.h, CVRecord.h, CodeViewError.h, RecordSerialization.h - maybe some of those aren't needed, if they're excessive includes or the like (they tend to seep in all over the place)) > > >> include/llvm/Object/COFF.h includes >>> include/llvm/DebugInfo/CodeView/CVDebugRecord.h >>> Also seems...

...nd on Object)? Any suggestions? Sink it into >> BinaryFormat or name a new library? >> >> (it's quite a few headers by the looks of it: CodeView.h, Line.h, >> SymbolRecord.h, CodeViewTypes.def, CodeViewSymbols.def, >> CodeViewRegisters.def, TypeIndex.h, CVRecord.h, CodeViewError.h, >> RecordSerialization.h - maybe some of those aren't needed, if they're >> excessive includes or the like (they tend to seep in all over the place)) >> >> >>> >>> >>>> include/llvm/Object/COFF.h includes >>>>> include/...

On Thu, Mar 22, 2018 at 12:55 PM Reid Kleckner <rnk at google.com> wrote: > On Wed, Mar 21, 2018 at 11:31 AM David Blaikie <dblaikie at gmail.com> wrote: > >> I'm looking at fixing some layering violations in LLVM & came across a >> few in the CodeView handling, specifically: >> >> lib/MC/MCCodeView includes several llvm/DebugInfo/CodeView headers

...uggestions? Sink it into >>> BinaryFormat or name a new library? >>> >>> (it's quite a few headers by the looks of it: CodeView.h, Line.h, >>> SymbolRecord.h, CodeViewTypes.def, CodeViewSymbols.def, >>> CodeViewRegisters.def, TypeIndex.h, CVRecord.h, CodeViewError.h, >>> RecordSerialization.h - maybe some of those aren't needed, if they're >>> excessive includes or the like (they tend to seep in all over the place)) >>> >>> >>>> >>>> >>>>> include/llvm/Object/COFF.h includes &...