<div dir="ltr">Hi Peter,<div><br></div><div>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).</div><div><br></div><div>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.</div><div><br></div><div>-eric</div></div><br><div class="gmail_quote"><div dir="ltr" class="gmail_attr">On Thu, Dec 19, 2019 at 7:34 AM Peter Waller <<a href="mailto:Peter.Waller@arm.com">Peter.Waller@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">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" rel="noreferrer" 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<br>
</blockquote></div>