[cfe-commits] TableGen: Goodbye RTTI!
Jakob Stoklund Olesen
stoklund at 2pi.dk
Wed Oct 10 10:28:07 PDT 2012
On Oct 9, 2012, at 8:53 PM, Chris Lattner <sabre at nondot.org> wrote:
>
> On Oct 8, 2012, at 9:10 AM, Sean Silva <silvas at purdue.edu> wrote:
>
>> This patch set removes the last traces of RTTI from TableGen (both in
>> LLVM and Clang) and removes the crutch for enabling RTTI when building
>> it.
>
> This all sounds fine to me, if it's ok with Jakob, go for it!
LGTM with these minor changes:
- The 'kind' enums should be protected, not public.
- The enumerator naming should follow the style guide.
- Don't use funky indentation in the enums.
/jakob
More information about the cfe-commits
mailing list