[PATCH] D20325: Add ARM cdp intrinsics

Renato Golin via cfe-commits cfe-commits at lists.llvm.org
Tue May 17 10:32:21 PDT 2016

rengolin added a comment.

In http://reviews.llvm.org/D20325#432086, @t.p.northover wrote:

> I don't think we need to wait until the document is public, necessarily. The entire AArch64 backend went in before the encodings were public outside ARM (except maybe in binutils source?) and releasing specifications with no implementation yet is a bit annoying too.

I don't think a small future ACLE builtin is on the same league as a whole new back-end. :)

Nor I think that having this when the docs are public, rather than now, will affect users in any way.

It's been our stance for a long time to require docs to approve changes, however small. I don't want to relax that which I think is a good constraint, not for such a seemly irrelevant issue.

I also doubt this will be the only addition in the new ACLE, so why not release the document, and then submit all changes then?

Or, maybe I am mistaken, and this is really that important... is it?




More information about the cfe-commits mailing list