On 2014/06/18 14:19:37, jbramley wrote:
https://codereview.chromium.org/335133002/diff/1/src/arm/cpu-arm.cc
File src/arm/cpu-arm.cc (right):
https://codereview.chromium.org/335133002/diff/1/src/arm/cpu-arm.cc#newcode46
src/arm/cpu-arm.cc:46: void __attribute__((naked)) CPU::FlushICache(void*
start,
size_t size) {
On 2014/06/18 14:08:31, ulan wrote:
> Do you know if this attribute is supported by clang? I can't find any
reference.
I couldn't find a definite source, but the following pages strongly hint
that
Clang supports it:
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=52646
http://comments.gmane.org/gmane.comp.compilers.clang.devel/29171
I also couldn't find reference to it (or any other __attribute__) in
Clang's
manual, but it's _not_ included in the list of GCC features that aren't
yet
supported:
http://clang.llvm.org/docs/UsersManual.html#gcc-extensions-not-implemented-yet
I was hoping that the try servers would have a Clang builder, but they
don't.
(I
don't have a Clang tool chain set up here.)
I can't get Ubuntu's 'clang' package to cross-compile V8 for ARM, but a
simple
test with __attribute__((naked)) works:
[test.c]
void __attribute__((naked)) test(void) {
__asm__ __volatile__("nop\n" : : : "memory");
}
https://codereview.chromium.org/335133002/
--
--
v8-dev mailing list
[email protected]
http://groups.google.com/group/v8-dev
---
You received this message because you are subscribed to the Google Groups "v8-dev" group.
To unsubscribe from this group and stop receiving emails from it, send an email
to [email protected].
For more options, visit https://groups.google.com/d/optout.
