[Diffusion] rL248439: set div/rem default values to 'expensive' in TargetTransformInfo's cost model

Sanjay Patel via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 14 14:42:39 PDT 2015


I don't see an official code owner (SimplifyCFG or TTI) either.

Hal provided the review for trunk. Hal, can you approve this for the 3.7
branch?

For reference, the next step to really fix the problem is here:
http://reviews.llvm.org/D13297


On Wed, Oct 14, 2015 at 1:34 PM, Dimitry Andric <dimitry at andric.com> wrote:

> dim added a subscriber: dim.
> dim added auditors: 3.7-release, llvm-commits, tstellarAMD, spatel.
> dim added a comment.
>
> This fixes PR24343 partially, and is required for FreeBSD on x86.  The
> long-term solution is still being worked on, but this commit itself should
> be merged into the 3.7 branch.
>
> Sanjay, I could not really find a code owner for this part of the tree,
> maybe you know whom to pick, or you can take the responsibility? :)
>
>
> Users:
>   spatel (Author, Auditor)
>   3.7-release (Auditor)
>   llvm-commits (Auditor)
>   tstellarAMD (Auditor)
>
> http://reviews.llvm.org/rL248439
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20151014/1ca6316c/attachment.html>


More information about the llvm-commits mailing list