[PATCH] D27309: builtins: Add ARM Thumb1 implementation for uidiv and uidivmod

Dimitry Andric via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Sat Mar 11 05:44:18 PST 2017


dim added a subscriber: strejda.
dim added a comment.

In https://reviews.llvm.org/D27309#698613, @rengolin wrote:

> Did someone revert this? I think we should until we're sure how to fix the problem.


In FreeBSD, @strejda committed this rather big hammer to work around it:  https://reviews.freebsd.org/rS314907, basically disabling all thumb support. :(

I would certainly not object to reverting https://reviews.llvm.org/rL288710 for now, and put up a new review to handle all concerns. @strejda, you already had a bunch of suggestions in your github commit, right?


Repository:
  rL LLVM

https://reviews.llvm.org/D27309





More information about the llvm-commits mailing list