This is an archive of the discontinued LLVM Phabricator instance.

Split off CUDA-specific Sema parts to a new file
ClosedPublic

Authored by eliben on Sep 2 2014, 3:57 PM.

Details

Reviewers
pcc
Summary

In line with SemaOpenMP.cpp, etc. CUDA-specific semantic analysis code goes into a separate file. This is in anticipation of adding extra functionality here in the near future.

No change in functionality.

Diff Detail

Event Timeline

eliben updated this revision to Diff 13185.Sep 2 2014, 3:57 PM
eliben retitled this revision from to Split off CUDA-specific Sema parts to a new file.
eliben updated this object.
eliben edited the test plan for this revision. (Show Details)
eliben added a reviewer: pcc.
eliben added a subscriber: Unknown Object (MLST).
pcc accepted this revision.Sep 2 2014, 6:17 PM
pcc edited edge metadata.

LGTM modulo nit.

lib/Sema/SemaCUDA.cpp
14

You should include clang/Sema/Sema.h first per the coding standards.

This revision is now accepted and ready to land.Sep 2 2014, 6:17 PM
eliben closed this revision.Sep 3 2014, 8:37 AM

Thanks for the quick review, Peter.

Committed in r217043.

lib/Sema/SemaCUDA.cpp
14

Done.