[llvm-dev] [RFC] Create llvm/lib/Frontend

Chris Lattner via llvm-dev llvm-dev at lists.llvm.org
Tue Nov 19 15:38:26 PST 2019



> On Nov 19, 2019, at 9:04 AM, Doerfert, Johannes <jdoerfert at anl.gov> wrote:
> 
>>> 
>>> First, I tried to place the OpenMP-IR-Builder into llvm/IR, right next
>>> to the llvm::IRBuilder, but it would soon introduce a dependence on
>>> other libraries (first TransformUtils) [2].
>> 
>> I think it is a great idea to share the OpenMP lowering code, but I’m
>> concerned about the name 'lib/Frontend’.  This is a very broad name
>> and there are lots of things that “could be useful for frontends” -
>> and a lot of definitions of what a “frontend" is.
> 
> Fair point. I'm open to suggestions wrt. the name. FWIW, the name was
> suggested because it was not only supposed to be OpenMP stuff (soon).

What are the other things?  Naming is often helped by having examples from the broader pool that will shape this.  It will also be helpful to understand what would be “in” vs “out” of scope for this over time.

> 
>> WDYT about naming it something like "lib/OpenMPGen” and
>> generalizing/renaming it later when the scope is more clear?
> 
> I can do that. It was also proposed to do "lib/Frontend/OpenMP" or
> something similar. Would that help or would you prefer not to have the
> top level "Frontend" until we actually move other things?

Sure,

-Chris
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20191119/b7ff1207/attachment.html>


More information about the llvm-dev mailing list