<div dir="ltr">Hi Richard,<div><br></div><div>Thanks for the updates! Let me know if there's anything you need in the rest of llvm to help this process :)</div><div><br></div><div>-eric</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Mon, Mar 23, 2020 at 9:32 AM Richard Barton <<a href="mailto:Richard.Barton@arm.com">Richard.Barton@arm.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">
<div lang="EN-GB">
<div class="gmail-m_8580160989201996022WordSection1">
<p class="MsoNormal">Hi Eric<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">I realised today that I never replied to this. Sorry about that.<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">You should see that we are not merging today, but when we do we’ll definitely give a few days of notice for this change. Our plan is to create a version of the merge in a fork of the llvm-project so that anyone interested is able to lay
their hands on this to see what things will look like after merging. This will contain the F18 history rewritten as a straightline branch, the merge commit to master and any other commits that we intend to submit at the same time.
<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">Also, we are not proposing to add flang to LLVM_ALL_PROJECTS from the start as it will have different compiler requirements for C++17. We also have no buildbots and no windows support so we feel that adding it to ALL_PROJECTS from the start
would potentially be disruptive to all. Instead we propose to add it in such a way that you have to opt in to build it. We have a patch up for review that does this here:
<a href="https://reviews.llvm.org/D72416" target="_blank">https://reviews.llvm.org/D72416</a>. Our intention is to commit this patch at the same time as merging and you’ll be able to see it in the dummy branch that we publish a few days before the merge.<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">Ta<u></u><u></u></p>
<p class="MsoNormal">Rich<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<div style="border-top:none;border-right:none;border-bottom:none;border-left:1.5pt solid blue;padding:0in 0in 0in 4pt">
<div>
<div style="border-right:none;border-bottom:none;border-left:none;border-top:1pt solid rgb(225,225,225);padding:3pt 0in 0in">
<p class="MsoNormal"><b><span lang="EN-US">From:</span></b><span lang="EN-US"> Eric Christopher <<a href="mailto:echristo@gmail.com" target="_blank">echristo@gmail.com</a>>
<br>
<b>Sent:</b> 17 March, 2020 00:33<br>
<b>To:</b> Richard Barton <<a href="mailto:Richard.Barton@arm.com" target="_blank">Richard.Barton@arm.com</a>><br>
<b>Cc:</b> <a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
<b>Subject:</b> Re: [llvm-dev] Upstreaming Flang - postponed to Monday 23rd March<u></u><u></u></span></p>
</div>
</div>
<p class="MsoNormal"><u></u> <u></u></p>
<div>
<p class="MsoNormal">Thanks for the update and your dedication to making sure everything is good to go before merge.<u></u><u></u></p>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">When you're ready would you mind sending a 24 and maybe even a 12 hour notice before merging? Those of us downstream are going to have our hands full :)<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">Thanks!<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">-eric<u></u><u></u></p>
</div>
</div>
<p class="MsoNormal"><u></u> <u></u></p>
<div>
<div>
<p class="MsoNormal">On Mon, Mar 16, 2020 at 1:55 AM Richard Barton via llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>> wrote:<u></u><u></u></p>
</div>
<blockquote style="border-top:none;border-right:none;border-bottom:none;border-left:1pt solid rgb(204,204,204);padding:0in 0in 0in 6pt;margin-left:4.8pt;margin-right:0in">
<p class="MsoNormal">Hi llvm-dev<br>
<br>
We have not been able to complete all the work we need to do before merging F18 into LLVM as Flang so we will not be dong that today as previously announced.<br>
<br>
We propose to slip this back a week to let us finish off the last bits of work. All code changes are in review as of Friday. If you want more detail, you can see the exact status here:
<a href="https://github.com/orgs/flang-compiler/projects/8" target="_blank">https://github.com/orgs/flang-compiler/projects/8</a><br>
<br>
Before we merge into the monorepo, we'll post a dummy version of the commit into a fork of llvm-project so folks can have a quick look at the code and history before we merge in.<br>
<br>
Sorry for the delay on this. We want to get it right before we merge into the monorepo.<br>
<br>
Ta<br>
Rich<br>
<br>
Richard Barton<br>
Principal Engineer - Arm Compiler for Linux<br>
Arm Manchester<br>
<br>
_______________________________________________<br>
LLVM Developers mailing list<br>
<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a><br>
<a href="https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev" target="_blank">https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev</a><u></u><u></u></p>
</blockquote>
</div>
</div>
</div>
</div>
</blockquote></div>