[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