[PATCH] D20325: Add ARM cdp intrinsics

[Renato Golin via cfe-commits]

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?

cheers,
--renato


Repository:
  rL LLVM

http://reviews.llvm.org/D20325