[PATCH] D20325: Add ARM cdp intrinsics

Ranjeet Singh via cfe-commits cfe-commits at lists.llvm.org
Wed May 18 02:23:55 PDT 2016


rs marked an inline comment as done.
rs added a comment.

> 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?


Thanks for reviewing.

It's not that important to have the intrinsic added before the document is released. I'll upload a new patch without the intrinsic in arm_acle.h and I'll move the assembly test to LLVM.


Repository:
  rL LLVM

http://reviews.llvm.org/D20325





More information about the cfe-commits mailing list