[PATCH] D31716: [Builtins] Fix div0 error in udivsi3

Saleem Abdulrasool via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 5 10:17:41 PDT 2017


compnerd requested changes to this revision.
compnerd added inline comments.
This revision now requires changes to proceed.


================
Comment at: lib/builtins/arm/udivsi3.S:189
 #      if defined(__ARM_EABI__)
+	push {r6, lr}
 	bl	__aeabi_idiv0 // due to relocation limit, can't use b.
----------------
Why `r6` and not `r7`?  That would be the FP, and saving that makes stack walking better.


https://reviews.llvm.org/D31716





More information about the llvm-commits mailing list