<p dir="ltr"><br>
On Mar 4, 2014 8:22 PM, "Chris Lattner" <<a href="mailto:clattner@apple.com">clattner@apple.com</a>> wrote:<br>
><br>
><br>
> On Mar 4, 2014, at 3:17 PM, Chandler Carruth <<a href="mailto:chandlerc@google.com">chandlerc@google.com</a>> wrote:<br>
><br>
>> On Tue, Mar 4, 2014 at 1:04 PM, Mark Seaborn <<a href="mailto:mseaborn@chromium.org">mseaborn@chromium.org</a>> wrote:<br>
>>><br>
>>> The PNaCl project has implemented various IR simplification passes that simplify LLVM IR by lowering complex features to simpler features.  We'd like to upstream some of these IR passes to LLVM.  We'd like to explore if this acceptable, and if so, how we should go about doing this.<br>

>><br>
>><br>
>> My question is somewhat different. I'm not questioning whether these are acceptable, I'm questioning why these are interesting and important for the LLVM project.<br>
><br>
><br>
> I share Chandler's concern.  If these aren't actively used by something in tree, they will bit rot.  The way to counter the bit rot would be to add extensive testcases... but that would just add an even larger burden on core LLVM developers to keep them up to date.<br>

><br>
> We have seen similar "obviously useful" pieces of infrastructure fall to the same fate (e.g., the C backend, which incidentally had very similar utilities back when it was alive).  Why would this be any different?<br>

><br>
My reading of the OP suggests that there are at least two projects that depend on these passes in production (and will for the foreseeable future). I wasn't around when the CBE was added; were there such users for it then? If not, then I would consider that a major difference in the situation.</p>

<p dir="ltr">-- Sean Silva</p>
<p dir="ltr">> -Chris<br>
><br>
><br>
> _______________________________________________<br>
> LLVM Developers mailing list<br>
> <a href="mailto:LLVMdev@cs.uiuc.edu">LLVMdev@cs.uiuc.edu</a>         <a href="http://llvm.cs.uiuc.edu">http://llvm.cs.uiuc.edu</a><br>
> <a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev">http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev</a><br>
><br>
</p>