<html><head><meta http-equiv="Content-Type" content="text/html charset=utf-8"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;" class=""><br class=""><div><blockquote type="cite" class=""><div class="">On May 15, 2015, at 12:20 PM, Jim Grosbach <<a href="mailto:grosbach@apple.com" class="">grosbach@apple.com</a>> wrote:</div><br class="Apple-interchange-newline"><div class="">Seems reasonable, especially since it’ll allow us to more effectively forward declare classes more.<br class=""><br class="">Please add a file-level comment blurb for MCInstrDesc.cpp and run clang-format on the file, but otherwise LGTM.<br class=""></div></blockquote>Committed with those changes, <span style="font-family: 'Helvetica Neue';" class="">r237478</span>. Thanks!</div><div><span style="font-family: 'Helvetica Neue'; line-height: normal;" class=""><br class=""></span></div><div><span style="font-family: 'Helvetica Neue'; line-height: normal;" class="">Pete</span></div><div><blockquote type="cite" class=""><div class=""><br class=""><br class=""><blockquote type="cite" class="">On May 13, 2015, at 3:54 PM, Pete Cooper <<a href="mailto:peter_cooper@apple.com" class="">peter_cooper@apple.com</a>> wrote:<br class=""><br class="">Hi Jim<br class=""><br class="">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.<br class=""><br class="">Thanks,<br class="">Pete<br class=""><br class=""><MCInstrDesc.diff><br class=""></blockquote><br class=""></div></blockquote></div><br class=""></body></html>