[clang-tools-extra] r261814 - check-clang-tools: Introduce the feature "target-headers".

[Alexander Kornienko via cfe-commits]

I've committed r261893 that adds stub headers and points the tool to the
right search directory. If the solution works, this commit can be reverted.

On Thu, Feb 25, 2016 at 2:12 AM, NAKAMURA Takumi via cfe-commits <
cfe-commits at lists.llvm.org> wrote:

> Author: chapuni
> Date: Wed Feb 24 19:12:57 2016
> New Revision: 261814
>
> URL: http://llvm.org/viewvc/llvm-project?rev=261814&view=rev
> Log:
> check-clang-tools: Introduce the feature "target-headers".
>
> For now, it just detects that host is non-Windows and target is msvc.
>
> FIXME: It should be probable for cross compilations. Detect whether
> target's headers would be available.
>
> Modified:
>
> clang-tools-extra/trunk/test/clang-tidy/modernize-deprecated-headers-cxx03.cpp
>
> clang-tools-extra/trunk/test/clang-tidy/modernize-deprecated-headers-cxx11.cpp
>     clang-tools-extra/trunk/test/lit.cfg
>
> Modified:
> clang-tools-extra/trunk/test/clang-tidy/modernize-deprecated-headers-cxx03.cpp
> URL:
> http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/clang-tidy/modernize-deprecated-headers-cxx03.cpp?rev=261814&r1=261813&r2=261814&view=diff
>
> ==============================================================================
> ---
> clang-tools-extra/trunk/test/clang-tidy/modernize-deprecated-headers-cxx03.cpp
> (original)
> +++
> clang-tools-extra/trunk/test/clang-tidy/modernize-deprecated-headers-cxx03.cpp
> Wed Feb 24 19:12:57 2016
> @@ -1,4 +1,5 @@
>  // RUN: %check_clang_tidy %s modernize-deprecated-headers %t -- --
> -std=c++03 -isystem %S/Inputs/Headers
> +// REQUIRES: target-headers
>
>  #include <assert.h>
>  #include <complex.h>
>
> Modified:
> clang-tools-extra/trunk/test/clang-tidy/modernize-deprecated-headers-cxx11.cpp
> URL:
> http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/clang-tidy/modernize-deprecated-headers-cxx11.cpp?rev=261814&r1=261813&r2=261814&view=diff
>
> ==============================================================================
> ---
> clang-tools-extra/trunk/test/clang-tidy/modernize-deprecated-headers-cxx11.cpp
> (original)
> +++
> clang-tools-extra/trunk/test/clang-tidy/modernize-deprecated-headers-cxx11.cpp
> Wed Feb 24 19:12:57 2016
> @@ -1,4 +1,5 @@
>  // RUN: %check_clang_tidy %s modernize-deprecated-headers %t -- --
> -std=c++11 -isystem %S/Inputs/Headers
> +// REQUIRES: target-headers
>
>  #include <assert.h>
>  #include <complex.h>
>
> Modified: clang-tools-extra/trunk/test/lit.cfg
> URL:
> http://llvm.org/viewvc/llvm-project/clang-tools-extra/trunk/test/lit.cfg?rev=261814&r1=261813&r2=261814&view=diff
>
> ==============================================================================
> --- clang-tools-extra/trunk/test/lit.cfg (original)
> +++ clang-tools-extra/trunk/test/lit.cfg Wed Feb 24 19:12:57 2016
> @@ -189,6 +189,12 @@ if not platform.system() in ['Windows']
>  if platform.system() not in ['Windows']:
>      config.available_features.add('ansi-escape-sequences')
>
> +# Set if default-target's system headers available.
> +# For example in general, MS headers are supposed unavailable on
> non-Windows hosts.
> +# FIXME: It should be probable for cross compilations.
> +if not (platform.system() not in ['Windows'] and re.match(r'.*-win32$',
> config.target_triple)):
> +    config.available_features.add('target-headers')
> +
>  check_clang_tidy = os.path.join(
>      config.test_source_root, "clang-tidy", "check_clang_tidy.py")
>  config.substitutions.append(
>
>
> _______________________________________________
> cfe-commits mailing list
> cfe-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20160225/c8f0daf5/attachment.html>