[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