[PATCH] D48915: [Sparc] Use the names .rem and .urem instead of __modsi3 and __umodsi3

Joerg Sonnenberger via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 9 07:33:15 PDT 2018


joerg added a comment.

Looking further, at least on NetBSD libgcc seems to always include both the "normal" and the .rem/.urem routines. While we currently don't replace __umodsi3 and friends, that looks more like an oversight on our part.
I asked the Sparc folks and they can't remember any special reason for why .urem should be used. I.e. it follows the normal Sparc ABI.
As such, I'm mostly ambivalent on this change and the rest of the block.


Repository:
  rL LLVM

https://reviews.llvm.org/D48915





More information about the llvm-commits mailing list