On Nov 14, 2009, at 6:25 AM, Mike Stump wrote:

> Author: mrs
> Date: Sat Nov 14 08:25:18 2009
> New Revision: 88792
> 
> URL: http://llvm.org/viewvc/llvm-project?rev=88792&view=rev
> Log:
> Add the name to the rtti data structure.
> 
> Modified:
>    cfe/trunk/lib/CodeGen/CGRtti.cpp
> 
> Modified: cfe/trunk/lib/CodeGen/CGRtti.cpp
> URL: 
> http://llvm.org/viewvc/llvm-project/cfe/trunk/lib/CodeGen/CGRtti.cpp?rev=88792&r1=88791&r2=88792&view=diff
> 
> ==============================================================================
> --- cfe/trunk/lib/CodeGen/CGRtti.cpp (original)
> +++ cfe/trunk/lib/CodeGen/CGRtti.cpp Sat Nov 14 08:25:18 2009
> @@ -15,7 +15,38 @@
> using namespace clang;
> using namespace CodeGen;
> 
> +class RttiBuilder {

Can RttiBuilder be in an anonymous namespace?  Also, should it be CGRTTI.cpp 
and RTTIBuilder since RTTI is an acronym?

-Chris
_______________________________________________
cfe-commits mailing list
[email protected]
http://lists.cs.uiuc.edu/mailman/listinfo/cfe-commits

Reply via email to