<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=Windows-1252">
</head>
<body>
Please apply one by one. There might be merge conflicts you’d have to resolve :) you can ping me on discord in case you get stuck.<br>
<br>
<br>
Aditya
<hr style="display:inline-block;width:98%" tabindex="-1">
<div id="divRplyFwdMsg" dir="ltr"><font face="Calibri, sans-serif" style="font-size:11pt" color="#000000"><b>From:</b> Ruijie Fang <ruijief@princeton.edu><br>
<b>Sent:</b> Wednesday, March 18, 2020 8:08:04 PM<br>
<b>To:</b> Aditya K <hiraditya@msn.com><br>
<b>Cc:</b> llvm-dev@lists.llvm.org <llvm-dev@lists.llvm.org><br>
<b>Subject:</b> Re: [llvm-dev] GSoC 2020 Project "Improve MegreFunctions to incorporate MergeSimilarFunctions patches and ThinLTO Support"</font>
<div> </div>
</div>
<div class="BodyFragment"><font size="2"><span style="font-size:11pt;">
<div class="PlainText">Hi Aditya,<br>
<br>
I'm starting to work on merging the patches --- One question, though,<br>
from someone who's still novice to the LLVM code review system:<br>
Should I merge your patches one by one into trunk, and then do a<br>
single "arc diff", or should I do multiple "arc patch" to first merge<br>
the five patches together (building on top of successive branches),<br>
and then all together into trunk, and then do an `arc diff`?<br>
<br>
Thanks,<br>
Ruijie<br>
<br>
On Wed, Mar 18, 2020 at 7:01 PM Aditya K <hiraditya@msn.com> wrote:<br>
><br>
> Hi Vishal, Ruijie,<br>
><br>
> Thanks for your interest in the project.<br>
> To get started, the first task would be to merge the 5 patches on top of trunk llvm.<br>
> The list of patches are listed in the project description: <a href="http://llvm.org/OpenProjects.html#llvm_mergesim">
http://llvm.org/OpenProjects.html#llvm_mergesim</a><br>
> Please create an account in llvm phabricator (reviews.llvm.org) if you haven't already, and put your patches there.<br>
><br>
> Let me know if you have further questions; both llvm-dev and discord are good places to reach out. My userid is: hiraditya<br>
><br>
> Welcome to GSoC!<br>
><br>
> Best,<br>
> -Aditya<br>
<br>
<br>
<br>
--<br>
RJF<br>
ruijief@princeton.edu<br>
<br>
<br>
On Wed, Mar 18, 2020 at 7:01 PM Aditya K <hiraditya@msn.com> wrote:<br>
><br>
> Hi Vishal, Ruijie,<br>
><br>
> Thanks for your interest in the project.<br>
> To get started, the first task would be to merge the 5 patches on top of trunk llvm.<br>
> The list of patches are listed in the project description: <a href="http://llvm.org/OpenProjects.html#llvm_mergesim">
http://llvm.org/OpenProjects.html#llvm_mergesim</a><br>
> Please create an account in llvm phabricator (reviews.llvm.org) if you haven't already, and put your patches there.<br>
><br>
> Let me know if you have further questions; both llvm-dev and discord are good places to reach out. My userid is: hiraditya<br>
><br>
> Welcome to GSoC!<br>
><br>
> Best,<br>
> -Aditya<br>
<br>
<br>
<br>
--<br>
RJF<br>
ruijief@princeton.edu<br>
</div>
</span></font></div>
</body>
</html>