This is an archive of the discontinued LLVM Phabricator instance.

[ASTImporter] Add test for C++'s try/catch statements.
ClosedPublic

Authored by teemperor on Aug 20 2018, 10:16 AM.

Diff Detail

Repository
rC Clang

Event Timeline

teemperor created this revision.Aug 20 2018, 10:16 AM
a_sidorin accepted this revision.Aug 20 2018, 1:45 PM
a_sidorin added a subscriber: a_sidorin.
a_sidorin added inline comments.
tools/clang-import-test/clang-import-test.cpp
201

Could you please add a newline after RTTI? Now, it looks like the comment is for all lines below.

This revision is now accepted and ready to land.Aug 20 2018, 1:45 PM
teemperor updated this revision to Diff 161580.Aug 20 2018, 3:11 PM
  • Clarified that the Exception options in LangOpts are not related to the dynamic_cast support by moving them above the comment (Thanks Aleksei!)
teemperor marked an inline comment as done.Aug 20 2018, 3:11 PM
This revision was automatically updated to reflect the committed changes.