[llvm-dev] CodeView layering

[Reid Kleckner via llvm-dev]

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
>   I guess MC could be made dependent on DebugInfoCodeView? But probably
> these things should be sunk into BinaryFormat as is the case for DWARF
> features used by MC?
>

I'd be OK with that. We could very easily introduce true link dependencies
on that library in the near future.


> include/llvm/Object/COFF.h includes
> include/llvm/DebugInfo/CodeView/CVDebugRecord.h
>   Also seems like this could/should/needs to be sunk into BinaryFormat?
>

These things are really more PE/PDB related than CodeView. They are in the
wrong place. I'd ask Saleem to help find a new home. Object or
BinaryFormats seem OK to me.


> I'm open to ideas & happy to do the work, or help in any way that might be
> useful.
>
> Thanks,
> - Dave
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20180322/69c4c560/attachment.html>