[PATCH] D22272: ARM: define __ARM_VFPV5__ when present.

2016-10-04 Thread Renato Golin via cfe-commits
rengolin accepted this revision. rengolin added a comment. This revision is now accepted and ready to land. Hi Tim, This patch seems stale. I think neither me nor Richard have any strong feelings against this, so I'll just approve and let you decide on which solution is best. I have a mild pref

Re: [PATCH] D22272: ARM: define __ARM_VFPV5__ when present.

2016-07-13 Thread Richard Barton via cfe-commits
richard.barton.arm added a subscriber: richard.barton.arm. richard.barton.arm added a comment. Hi all At first glance I thought this would be something that is in the ACLE, so was going to give an opinion. On second look, this is not actually an ACLE macro, and seems to be a clang special - my

Re: [PATCH] D22272: ARM: define __ARM_VFPV5__ when present.

2016-07-12 Thread Renato Golin via cfe-commits
rengolin added a reviewer: kristof.beyls. rengolin added a comment. I don't mind either way, but would be good to be compatible with gcc in that respect. Least surprise and all. ARM was naming it fpv8 for the command line options and other things, might also be good to keep the consistency. Wo

[PATCH] D22272: ARM: define __ARM_VFPV5__ when present.

2016-07-12 Thread Tim Northover via cfe-commits
t.p.northover created this revision. t.p.northover added reviewers: rengolin, jmolloy. t.p.northover added a subscriber: cfe-commits. Herald added subscribers: mcrosier, rengolin, aemerson. We've had embedded developers requesting we extend the __ARM_VFPVn__ series to support Cortex-M7, which sou