[PATCHES] A module inliner pass with a greedy call site queue

[Yin Ma]

Hi Chandler,

 

I am not familiar how to do consensus thing for merging. What is the good

way to do it? 

 

Thanks,

 

Yin 

 

From: Chandler Carruth [mailto:chandlerc at google.com] 
Sent: Tuesday, August 12, 2014 1:19 PM
To: Yin Ma
Cc: Jiangning Liu; Gerolf Hoflehner; Nick Lewycky; James Molloy; Hal Finkel; Commit Messages and Patches for LLVM
Subject: Re: [PATCHES] A module inliner pass with a greedy call site queue

 

 

On Tue, Aug 12, 2014 at 1:16 PM, Yin Ma <yinma at codeaurora.org <mailto:yinma at codeaurora.org> > wrote:

Hi Everybody,

 

It has been several weeks after I submitted this greedy inliner patch. Some people like 

and some hesitate. This inliner has shown promising potentials in various situations and

default off. I would like to know whether we can merge this into LLVM at first so the people 

who like it can continue tuning it up? For people who still has question about what is the 

right inliner, I am about to submit a BOF proposal to discuss this issue in the coming LLVM

dev conference.  How do you guys think?


I don't think we should merge this until there is some consensus. I think the BOF is already moving in the right direction to try to find it.

-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20140813/b656d867/attachment.html>