<div dir="ltr">Thanks Aditya and Vishal for the context on the patches. I'm glad to see this work moving forward.<div>Teresa</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Sat, Mar 21, 2020 at 11:24 AM Aditya K <<a href="mailto:hiraditya@msn.com">hiraditya@msn.com</a>> wrote:<br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left:1px solid rgb(204,204,204);padding-left:1ex">hi Teresa,<br>
This was the starter assignment for the students to merge the patches.<br>
We'll address the comments from the original patches once the intern has been finalized.<br>
JF Bastien has agreed to co-mentor the project, we'll try to address the core issues to make merge-function as robust as possible. <br>
<br>
Thanks,<br>
-Aditya<br>
<br>
<br>
<br>
<br>
<br>
<br>
<br>
From: Teresa Johnson <<a href="mailto:tejohnson@google.com" target="_blank">tejohnson@google.com</a>><br>
<br>
Sent: Friday, March 20, 2020 1:42 PM<br>
To: Aditya K <<a href="mailto:hiraditya@msn.com" target="_blank">hiraditya@msn.com</a>>; JF Bastien <<a href="mailto:jfbastien@apple.com" target="_blank">jfbastien@apple.com</a>><br>
Cc: llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>>; <a href="mailto:chebroluvishal@gmail.com" target="_blank">chebroluvishal@gmail.com</a> <<a href="mailto:chebroluvishal@gmail.com" target="_blank">chebroluvishal@gmail.com</a>>; <a href="mailto:ruijief@princeton.edu" target="_blank">ruijief@princeton.edu</a> <<a href="mailto:ruijief@princeton.edu" target="_blank">ruijief@princeton.edu</a>><br>
Subject: Re: [llvm-dev] GSoC 2020 Project "Improve MegreFunctions to incorporate MergeSimilarFunctions patches and ThinLTO Support"<br>
<br>
Hi all, I see there are a lot of new patches being produced to merge various combinations of the original patches together and rebase with head. I just added myself a reviewer to the ones I saw. A couple high level comments: These various mergings<br>
 of the patches together don't need to all be separate patches in Phabricator. Really only whatever you want reviewed by the community needs to go up on Phabricator, which is presumably the final merged and rebase patch.  Also, please keep all the original<br>
 reviewers on the new patch(es). There are a number of comments on the original patches that still need to be resolved fyi, so please take a look at those before proceeding too far. +JF<br>
 Bastien who also commented to that effect. It might be good to coordinate your efforts as well.<br>
<br>
Thanks!<br>
Teresa<br>
<br>
</blockquote></div><br clear="all"><div><br></div>-- <br><div dir="ltr" class="gmail_signature"><div dir="ltr"><div><span style="font-family:Times;font-size:medium"><table cellspacing="0" cellpadding="0"><tbody><tr style="color:rgb(85,85,85);font-family:sans-serif;font-size:small"><td nowrap style="border-top:2px solid rgb(213,15,37)">Teresa Johnson |</td><td nowrap style="border-top:2px solid rgb(51,105,232)"> Software Engineer |</td><td nowrap style="border-top:2px solid rgb(0,153,57)"> <a href="mailto:tejohnson@google.com" target="_blank">tejohnson@google.com</a> |</td><td nowrap style="border-top:2px solid rgb(238,178,17)"><br></td></tr></tbody></table></span></div></div></div>