[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.
More information about the cfe-commits