[cfe-commits] [PATCH] Add PNaCl ABIInfo

Chandler Carruth chandlerc at google.com
Fri Sep 23 00:41:15 PDT 2011


On Thu, Sep 22, 2011 at 11:59 PM, John McCall <rjmccall at apple.com> wrote:

> > While our ABI is not 100% finalized, until this patch is committed, we
> > can't do any external testing with Clang. (unless we create our own
> > branch, which we'd rather not)
>
> I'm sorry, but we do expect code to go through code review, which
> includes a thorough review of the underlying motivations and technical
> design.


I also don't see why this would block testing. There are lots of ways to
patch something together if you just want to get feedback on what you're
doing from users... That might actually provide good data for the very
codereview and design discussion taking place. It's not like external
parties can start using what you have in production environments or regular
development processes if the very ABI is still in flux, but maybe I don't
understand something.

If you just want a way to share what you're working on, toss it up on github
with the patch applied and point people at it. There are lots of ways of
sharing code during the review process.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20110923/996a3987/attachment.html>


More information about the cfe-commits mailing list