<div dir="ltr">I know we spoke on IRC and at the dev meeting, but a good email here helps too. I've added Hal and Iain explicitly here since we've had talks with both of them about this.<div><br></div><div>For the record: I'm in favor of this. </div><div><br></div><div>Thanks!</div><div><br></div><div>-eric<br><br><div class="gmail_quote"><div dir="ltr">On Mon, Aug 20, 2018 at 12:42 PM Kit Barton via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div dir="ltr">Hi all,<div>I just posted two patches on phabricator to remove Darwin support from the POWER backend. </div><div><br></div><div>I thought I had posted this on the dev mailing list some time ago, but I cannot find a reference to it now. </div><div><br></div><div>The intention here is to cleanup the POWER backend. Darwin has not been supported on POWER for quite some time now, and we'd like to use this as an opportunity to clean up several aspects of the backend code. If there is any concerns with this, please let me know.</div><div><br></div><div>As for the approach, the patch that I posted will issue an error message from the backend if you attempt to use the Darwin ABI on PPC. The intention is that once this patch lands, all of the Darwin-specific code paths will essentially be dead-code, and we can remove them on demand as we're touching various pieces of code. This cleanup should be done in stand-alone patches, but can be done as NFC changes and thus reviewed post-commit, whenever possible. </div><div><br></div><div>I've cleaned up all of the test cases effected by this, mostly by converting them to a different powerpc target so as to not lose test coverage. There were a few tests that were ppc-darwin specific, which were removed.</div><div><br></div><div>There is a companion patch to clean up test cases in Clang as well. I think we can use a similar approach for cleaning up ppc-specific darwin code in Clang, although I don't have a good idea right now for how much cleanup that will be.</div><div><br></div><div>Thanks.</div><div><br></div></div>
_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
<a href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" rel="noreferrer" target="_blank">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><br>
</blockquote></div></div></div>