<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body text="#000000" bgcolor="#FFFFFF">
<br>
<div class="moz-cite-prefix">On 08/20/2018 02:42 PM, Kit Barton via
llvm-dev wrote:<br>
</div>
<blockquote type="cite"
cite="mid:CAODshGC_-1et6eiXB+5A4DQrK4nfC0KC2-aSBbqoKEmAcJ7a7Q@mail.gmail.com">
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
<div dir="ltr">Hi all,
<div>I just posted two patches on phabricator to remove Darwin
support from the POWER backend. <br>
</div>
</div>
</blockquote>
<br>
Thanks, Kit.<br>
<br>
To connect the two threads, the last RFC we had on this subject was
here:
<a class="moz-txt-link-freetext" href="http://lists.llvm.org/pipermail/llvm-dev/2016-October/106359.html">http://lists.llvm.org/pipermail/llvm-dev/2016-October/106359.html</a> -
it looks like we're all now in agreement to move forward.<br>
<br>
<blockquote type="cite"
cite="mid:CAODshGC_-1et6eiXB+5A4DQrK4nfC0KC2-aSBbqoKEmAcJ7a7Q@mail.gmail.com">
<div dir="ltr">
<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. <br>
</div>
</div>
</blockquote>
<br>
This is reasonable. Please proceed.<br>
<br>
-Hal<br>
<br>
<blockquote type="cite"
cite="mid:CAODshGC_-1et6eiXB+5A4DQrK4nfC0KC2-aSBbqoKEmAcJ7a7Q@mail.gmail.com">
<div dir="ltr">
<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>
<fieldset class="mimeAttachmentHeader"></fieldset>
<br>
<pre wrap="">_______________________________________________
LLVM Developers mailing list
<a class="moz-txt-link-abbreviated" href="mailto:llvm-dev@lists.llvm.org">llvm-dev@lists.llvm.org</a>
<a class="moz-txt-link-freetext" href="http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev">http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a>
</pre>
</blockquote>
<br>
<pre class="moz-signature" cols="72">--
Hal Finkel
Lead, Compiler Technology and Programming Languages
Leadership Computing Facility
Argonne National Laboratory</pre>
</body>
</html>