This is an archive of the discontinued LLVM Phabricator instance.

Clean up some existing keyword tests in the test/Lexer directory
ClosedPublic

Authored by ygao on Apr 22 2014, 3:36 PM.

Details

Summary

Hi,
I am trying to clean up the existing keyword tests in the test/Lexer directory by using the
pre-defined __is_identifier() macro:

  1. Merge the contents of cxx0x_keyword.cpp and clang-keywords.cpp into a new file keywords_test.cpp
  2. Remove gnu_keywords.c because the contents of this file is already merged into keywords_test.c

Can someone take a look whether this change looks reasonable?

  • Gao

Diff Detail

Event Timeline

ygao updated this revision to Diff 8750.Apr 22 2014, 3:36 PM
ygao retitled this revision from to Clean up some existing keyword tests in the test/Lexer directory.
ygao updated this object.
ygao edited the test plan for this revision. (Show Details)
ygao updated this revision to Diff 8756.Apr 22 2014, 5:31 PM
ygao updated this revision to Diff 8759.Apr 22 2014, 5:45 PM
ygao added a subscriber: Unknown Object (MLST).
ygao added a comment.May 5 2014, 2:45 PM

A gentle ping?

rsmith accepted this revision.May 5 2014, 3:04 PM
rsmith added a reviewer: rsmith.
rsmith added a subscriber: rsmith.

FYI, Phabricator doesn't send mail when you change the CC's on a review, so no-one was informed that this change existed until just now.

LGTM, thanks!

test/Lexer/keywords_test.cpp
4–5

It might be nice to also have an IS_TYPE, and check there that tentative parsing knows that the keyword is a type. Something like:

#define IS_TYPE(NAME) void is_##TYPE##_type() { int(f(TYPE)); } // ok, function declaration
This revision is now accepted and ready to land.May 5 2014, 3:04 PM
ygao closed this revision.May 6 2014, 7:06 PM