[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