<div dir="ltr">Great, thanks for the update! <br></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Dec 19, 2019 at 9:11 AM Stephen Scalpone <<a href="mailto:sscalpone@nvidia.com">sscalpone@nvidia.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-US">
<div class="gmail-m_8775849095015702609WordSection1">
<p class="MsoNormal">Hi Eric,<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">The LICENSE file in the f18 github repo reflects the new llvm licensing. We expect to change the file headers to reflect the relicensing in the next few days. The NVIDIA copyright will be removed.<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<p class="MsoNormal">- Steve<u></u><u></u></p>
<p class="MsoNormal"><u></u> <u></u></p>
<div style="border-right:none;border-bottom:none;border-left:none;border-top:1pt solid rgb(181,196,223);padding:3pt 0in 0in">
<p class="MsoNormal"><b><span style="font-size:12pt;color:black">From: </span></b><span style="font-size:12pt;color:black">Eric Christopher <<a href="mailto:echristo@gmail.com" target="_blank">echristo@gmail.com</a>><br>
<b>Date: </b>Thursday, December 19, 2019 at 8:56 AM<br>
<b>To: </b>Peter Waller <<a href="mailto:Peter.Waller@arm.com" target="_blank">Peter.Waller@arm.com</a>>, Chris Lattner <<a href="mailto:clattner@nondot.org" target="_blank">clattner@nondot.org</a>><br>
<b>Cc: </b>llvm-dev <<a href="mailto:llvm-dev@lists.llvm.org" target="_blank">llvm-dev@lists.llvm.org</a>>, Hal Finkel <<a href="mailto:hfinkel@anl.gov" target="_blank">hfinkel@anl.gov</a>>, Stephen Scalpone <<a href="mailto:sscalpone@nvidia.com" target="_blank">sscalpone@nvidia.com</a>>, Chandler Carruth <<a href="mailto:chandlerc@gmail.com" target="_blank">chandlerc@gmail.com</a>><br>
<b>Subject: </b>Re: [llvm-dev] Flang landing in the monorepo<u></u><u></u></span></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">Hi Peter, <u></u><u></u></p>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">At this point I'm very confused at the point of landing the code. Outside of the flang name there doesn't appear to be a single thing that says this is an llvm project. There's no evidence of llvm code or style guide review, no use of llvm
APIs, or design similar to existing front ends. In addition, the license files also don't appear to match the current license of the project - this appears to be nvidia copyright (though under apache 2 as is correct as far as I can tell).<u></u><u></u></p>
</div>
<div>
<p class="MsoNormal"><u></u> <u></u></p>
</div>
<div>
<p class="MsoNormal">I'd be very curious in reading the minutes from the board session where this was discussed and what conditions were given for commit to the repository.<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 Thu, Dec 19, 2019 at 7:34 AM Peter Waller <<a href="mailto:Peter.Waller@arm.com" target="_blank">Peter.Waller@arm.com</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">On 18/12/2019 21:49, Eric Christopher wrote:<br>
> Yes, I looked through those sources and a number of my questions <br>
> around which clang versions have been supported and directory <br>
> structure. I think the only difference is removing the direct <br>
> questions about earlier flang, but I still don't see code generation <br>
> or uses of llvm libraries that would conform to "written in the style <br>
> of LLVM and clang". Can you perhaps point me to where I'm missing <br>
> these things?<br>
I can't speak with authority on all of these issues.<br>
<br>
In terms of clang versions, I understand that clang version 7 and 8 are <br>
currently supported. We would expect it to work with newer LLVM <br>
versions, and<br>
the readme is currently out of date. The intent is that it will work <br>
with all branches of LLVM and the community will build up CI to protect <br>
this.<br>
<br>
It's worth mentioning that merging flang in at this point does not <br>
affect the existing LLVM build in any way. Patches to integrate the <br>
build system are expected in the near future, and be subject to the <br>
normal LLVM code review processes.<br>
<br>
I understand that code generation is a work in progress and is expected <br>
to start landing in the not too distant future. Other people (Steve <br>
Scalpone, cc'd, and others) can perhaps speak to this more than me.<br>
<br>
In terms of using LLVM ADTs, etc, I expect that once flang is part of <br>
the monorepo, there will be a greater usage of those things.<br>
> Chris's earlier acceptance aside I don't see any evidence of code <br>
> review as part of that and so I'd expect we'd see more here.<br>
Code review has been happening all along in the the f18 github repository:<br>
<a href="https://github.com/flang-compiler/f18/pulls" target="_blank">https://github.com/flang-compiler/f18/pulls</a><br>
<br>
I am operating on the assumption that the code will land with no <br>
additional review by members of the LLVM community, and future code <br>
review will happen with the same mechanisms that the wider LLVM project use.<br>
<br>
Regards,<br>
<br>
- Peter<u></u><u></u></p>
</blockquote>
</div>
</div>
<div>
<hr>
</div>
<div>This email message is for the sole use of the intended recipient(s) and may
contain confidential information. Any unauthorized review, use, disclosure
or distribution is prohibited. If you are not the intended recipient,
please contact the sender by reply email and destroy all copies of the original
message. </div>
<div>
<hr>
</div>
</div>
</blockquote></div>