Add support for AArch64 v8.1 architecture. Briefly it is described on
http://community.arm.com/groups/processors/blog/2014/12/02/the-armv8-a-architecture-and-its-ongoing-development
Depends on http://reviews.llvm.org/D8766
Details
Details
Diff Detail
Diff Detail
Event Timeline
Comment Actions
Hi Vlad,
We don't tend to add "generic" CPUs like this. What's wrong with using "-mcpu=cortex-a57+v8.1a" ?
Cheers,
James
Comment Actions
It has been added already in backend - http://reviews.llvm.org/D8505 , where we got no other decent possibility.
Why not to use the same in frontend?
Also, see comments below
lib/Basic/Targets.cpp | ||
---|---|---|
4249 | for a57+v8.1a it would be weird to try to pass the test, commented below. __ARM_ARCH_8A__ 1 But I suspect we'd need to have __ARM_ARCH_8_1A__ 1 , as tested in very last test here. | |
test/Preprocessor/arm-target-features.c | ||
322 | for a57+v8.1a it would be weird to pass this test |
for a57+v8.1a it would be weird to try to pass the test, commented below.
a57 would normally normally yield macro
see https://github.com/llvm-mirror/clang/blob/1d941335617c6199e8340084c8a6c37c1b8014ec/lib/Basic/Targets.cpp#L4244
But I suspect we'd need to have
, as tested in very last test here.
Unfortunately, without ACLE issued we could not yet realize, whether we'd need both of them.