[PATCH] Move McInstrDesc methods which use MCInst to their own cpp file
Pete Cooper
peter_cooper at apple.com
Fri May 15 14:34:47 PDT 2015
> On May 15, 2015, at 12:20 PM, Jim Grosbach <grosbach at apple.com> wrote:
>
> Seems reasonable, especially since it’ll allow us to more effectively forward declare classes more.
>
> Please add a file-level comment blurb for MCInstrDesc.cpp and run clang-format on the file, but otherwise LGTM.
Committed with those changes, r237478. Thanks!
Pete
>
>
>> On May 13, 2015, at 3:54 PM, Pete Cooper <peter_cooper at apple.com> wrote:
>>
>> Hi Jim
>>
>> This will become useful in a later commit to reduce the includes in MC headers. MCInstrDesc.h currently includes MCInst.h (then transitively most of the MC headers) because of 2 methods. This patch moves these out to their own cpp file.
>>
>> Thanks,
>> Pete
>>
>> <MCInstrDesc.diff>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150515/47994926/attachment.html>
More information about the llvm-commits
mailing list