Conversation
Also update the OpenMP text to state that it is used by Flang.
index.html
Outdated
| sort of tools that can be built using the Clang frontend as a library to | ||
| parse C/C++ code.</p></li> | ||
|
|
||
| <li><p><b><a href="https://flang.llvm.org">Flang</a></b> is a modern "LLVM |
There was a problem hiding this comment.
Tiny nit, but I'd consider dropping scare quotes; perhaps even modern LLVM-based [compiler] instead of "LLVM native" [compiler] would be more straightforward?
There was a problem hiding this comment.
I intentionally used the quotes since the description for clang uses them too. Perhaps I could just say "Fortran is a modern Fortran compiler ..." and leave it at that.
What do you think?
There was a problem hiding this comment.
Interesting, haven't noticed that! Yeah, perhaps a modern Fortran compiler works, too.
I did see LLVM-based in a couple of different contexts, https://github.com/search?q=repo%3Allvm%2Fllvm-www%20llvm-based&type=code, so it seems natural, too. I guess it depends on whether we'd like to emphasize this aspect.
bonachea
left a comment
There was a problem hiding this comment.
Thanks @tarunprabhu for pursuing this important improvement to flang visibility!
Should we also consider similarly updating the "LLVM Features" page? It currently does mention Fortran in one place, but this could potentially be amplified somewhat (particularly under the incomplete list of frontends under "Strengths of the LLVM System").
It might be better to update the existing contents of that page before adding more Fortran/Flang-specific information to it. For instance, this bullet point in "Strengths of the LLVM System"
contains broken links and references frontends for Java and Scheme that I don't think are part of the llvm-project. |
Also update the OpenMP text to state that it is used by Flang.
@tlattner, could you take a look, please? I have never contributed to this repo, so please let me know if I was supposed to follow a different process to get this text added.