[PATCH] D14115: Fix Android clear_cache ASM
angelsl via llvm-commits
llvm-commits at lists.llvm.org
Tue Oct 27 08:13:51 PDT 2015
angelsl added a comment.
In http://reviews.llvm.org/D14115#276198, @compnerd wrote:
> This seems incorrect. There is an argument to the call. Why do you think that this change is needed?
.../build/src/compiler-rt/lib/builtins/clear_cache.c: In function '__clear_cache':
.../build/src/compiler-rt/lib/builtins/clear_cache.c:106:9: error: asm-specifier for variable 'start_reg' conflicts with asm clobber list
__asm __volatile("svc 0x0" : "=r"(start_reg)
^
.../build/src/compiler-rt/lib/builtins/clear_cache.c:106:9: error: asm-specifier for variable 'start_reg' conflicts with asm clobber list
.../build/obj/arm-linux-androideabi/rt/compiler-rt/triple/builtins/armv7/SubDir.lib__builtins/clear_cache.o] Error 1
I believe the compiler takes into account `r0` being changed/clobbered, since it's already in the output register list.
http://reviews.llvm.org/D14115
More information about the llvm-commits
mailing list