[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