[PATCH] D46836: Fix some rtti-options tests

2018-05-15 Thread Phabricator via Phabricator via cfe-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rC332384: Fixed some rtti-options tests. (authored by ssrivastava, committed by ). Changed prior to commit: https://reviews.llvm.org/D46836?vs=146632=146893#toc Repository: rC Clang

[PATCH] D46836: Fix some rtti-options tests

2018-05-15 Thread Filipe Cabecinhas via Phabricator via cfe-commits
filcab accepted this revision. filcab added a comment. This revision is now accepted and ready to land. LGTM Comment at: test/Driver/rtti-options.cpp:13 // RUN: %clang -### -c -fno-rtti -frtti %s 2>&1 | FileCheck -check-prefix=CHECK-RTTI %s -// RUN: %clang -### -c -frtti

[PATCH] D46836: Fix some rtti-options tests

2018-05-14 Thread Sunil Srivastava via Phabricator via cfe-commits
Sunil_Srivastava created this revision. Herald added a subscriber: cfe-commits. Certain tests in rtti-options.cpp are not really testing anything because they are testing for the absence of -frtti option to the cc1 process. Since the cc1 process does not take -frtti option, these tests are