[cfe-dev] Heads up, renaming attempt #2!

[John McCall]

On Dec 21, 2012, at 1:18 PM, Chandler Carruth <chandlerc at gmail.com> wrote:
> So, last time I tried, I needed a bit more tooling in place to do the renaming sanely. =] That's taken care of, and so I'm hoping to do the renaming from 'clang/lib/CodeGen/...' to 'clang/lib/IRGen/...' over the holidays (hopefully minimal patches outstanding).
> 
> I'm also going to try to systematically clean up the naming of files and classes.
> 
> The current leading strategy for files:
> 
>   lib/CodeGen/CGCall.cpp -> lib/IRGen/Call.cpp
>   lib/CodeGen/CodeGenFunction.cpp -> lib/IRGen/Function.cpp
> 
> The current leading strategy for classes:
> 
>   clang::CGFoo -> clang::irgen::Foo
>   clang::CodeGenBar -> clang::irgen::Bar
> 
> I'm asking Doug and John to paint this bikeshed so that we don't just go around in circles endlessly.

I really like having a prefix on file names so that they aren't immediately confusable — visually, by IDEs, by C-x b, etc. — with files from other parts of the compiler.  For example, I think it would be a real loss if there were three Decl.cpp files in clang: one in AST, one in Sema, and one in IRGen.

For lack of a better idea, let's go with replacing everything that's "CodeGen" with "IRGen" and "CG" with "IR".  "IR" has a pretty unambiguous meaning in the context of LLVM.

Everything in CodeGen is already in a namespace, "CodeGen".  This should be renamed to "irgen".

Thanks for doing this!

John.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-dev/attachments/20121221/e8b544cb/attachment.html>