<html><head><meta http-equiv="Content-Type" content="text/html charset=windows-1252"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><div>I normally expect a feature to be a standalone bit of functionality — like a new optimization pass or some refactoring work. The openmp work is much bigger in nature. How many more patches will you need to commit? Are the features self contained (i.e., it doesn’t modify other parts of the compiler)?</div><div><br></div><div>-bw</div><br><div><div>On Jul 22, 2014, at 10:32 AM, Jack Howarth <<a href="mailto:howarth.mailing.lists@gmail.com">howarth.mailing.lists@gmail.com</a>> wrote:</div><br class="Apple-interchange-newline"><blockquote type="cite"><div dir="ltr">Bill,<div> Can you clarify on 'existing feature'? Does this allow for addition of missing functionality for the openmp support such as…</div><div><br></div><div><a href="http://llvm.org/viewvc/llvm-project?view=revision&revision=213639">http://llvm.org/viewvc/llvm-project?view=revision&revision=213639</a><br>
</div><div><br></div><div>Almost all of the remaining commits for clang-omp support will be implementing missing functionality in the openmp support.</div><div> Jack</div></div><div class="gmail_extra"><br><br>
<div class="gmail_quote">On Tue, Jul 22, 2014 at 12:50 PM, Bill Wendling <span dir="ltr"><<a href="mailto:isanbard@gmail.com" target="_blank">isanbard@gmail.com</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex">
Hi Developers,<br>
<br>
This is to clarify which patches may be merged into the 3.5 branch.<br>
<br>
* All doc changes may go in. In fact, you’re encouraged to update the branch's ReleaseNotes.rst file! :-)<br>
<br>
* All non-documentation patches should first receive approval from the appropriate code maintainer. (A blanket approval is acceptable for the first phase of testing.)<br>
<br>
* If your patch fixes a major bug, it may go in.<br>
<br>
* If your patch is to complete an *existing* feature, then it may go into the 3.5 branch. However, this feature must be completed by Phase II of testing.<br>
<br>
Once Phase II testing starts, we will be accepting only those patches that fix major bugs.<br>
<br>
Share and enjoy!<br>
-bw<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/" target="_blank">http://llvm.cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev</a><br>
</blockquote></div><br></div>
</blockquote></div><br></body></html>