[llvm-branch-commits] [clang-tools-extra-branch] r352368 - Merging r352231:

[Hans Wennborg via llvm-branch-commits]

Author: hans
Date: Mon Jan 28 06:13:58 2019
New Revision: 352368

URL: http://llvm.org/viewvc/llvm-project?rev=352368&view=rev
Log:
Merging r352231:
Redirecting to URL 'https://llvm.org/svn/llvm-project/clang-tools-extra/trunk':
------------------------------------------------------------------------
r352231 | jonastoth | 2019-01-25 20:05:12 +0100 (Fri, 25 Jan 2019) | 13 lines

[clang-tidy] fix unit tests for dropped _Float16 support in X86

Summary:
Because _Float16 was disabled for X86 targets the unit-tests started failing.
Extract the pieces for _Float16 and run theses tests under AArch64.

Reviewers: aaron.ballman, erichkeane, lebedev.ri

Reviewed By: erichkeane

Subscribers: javed.absar, xazax.hun, kristof.beyls, cfe-commits

Differential Revision: https://reviews.llvm.org/D57249
------------------------------------------------------------------------

Added:
    clang-tools-extra/branches/release_80/test/clang-tidy/readability-uppercase-literal-suffix-float16.cpp
      - copied unchanged from r352231, clang-tools-extra/trunk/test/clang-tidy/readability-uppercase-literal-suffix-float16.cpp
Modified:
    clang-tools-extra/branches/release_80/   (props changed)
    clang-tools-extra/branches/release_80/test/clang-tidy/readability-uppercase-literal-suffix-floating-point.cpp
    clang-tools-extra/branches/release_80/test/clang-tidy/readability-uppercase-literal-suffix-hexadecimal-floating-point.cpp

Propchange: clang-tools-extra/branches/release_80/
------------------------------------------------------------------------------
--- svn:mergeinfo (original)
+++ svn:mergeinfo Mon Jan 28 06:13:58 2019
@@ -1 +1 @@
-/clang-tools-extra/trunk:351463,351466-351468,351531,351686,351738,351788,352040
+/clang-tools-extra/trunk:351463,351466-351468,351531,351686,351738,351788,352040,352231

Modified: clang-tools-extra/branches/release_80/test/clang-tidy/readability-uppercase-literal-suffix-floating-point.cpp
URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/branches/release_80/test/clang-tidy/readability-uppercase-literal-suffix-floating-point.cpp?rev=352368&r1=352367&r2=352368&view=diff
==============================================================================
--- clang-tools-extra/branches/release_80/test/clang-tidy/readability-uppercase-literal-suffix-floating-point.cpp (original)
+++ clang-tools-extra/branches/release_80/test/clang-tidy/readability-uppercase-literal-suffix-floating-point.cpp Mon Jan 28 06:13:58 2019
@@ -97,34 +97,6 @@ void floating_point_suffix() {
   static constexpr auto v13 = 1.e0Q; // OK.
   static_assert(is_same<decltype(v13), const __float128>::value, "");
   static_assert(v13 == 1., "");
-
-  // _Float16
-
-  static constexpr auto v14 = 1.f16;
-  // CHECK-MESSAGES: :[[@LINE-1]]:31: warning: floating point literal has suffix 'f16', which is not uppercase
-  // CHECK-MESSAGES-NEXT: static constexpr auto v14 = 1.f16;
-  // CHECK-MESSAGES-NEXT: ^ ~
-  // CHECK-MESSAGES-NEXT: {{^ *}}F16{{$}}
-  // CHECK-FIXES: static constexpr auto v14 = 1.F16;
-  static_assert(is_same<decltype(v14), const _Float16>::value, "");
-  static_assert(v14 == 1.F16, "");
-
-  static constexpr auto v15 = 1.e0f16;
-  // CHECK-MESSAGES: :[[@LINE-1]]:31: warning: floating point literal has suffix 'f16', which is not uppercase
-  // CHECK-MESSAGES-NEXT: static constexpr auto v15 = 1.e0f16;
-  // CHECK-MESSAGES-NEXT: ^ ~
-  // CHECK-MESSAGES-NEXT: {{^ *}}F16{{$}}
-  // CHECK-FIXES: static constexpr auto v15 = 1.e0F16;
-  static_assert(is_same<decltype(v15), const _Float16>::value, "");
-  static_assert(v15 == 1.F16, "");
-
-  static constexpr auto v16 = 1.F16; // OK.
-  static_assert(is_same<decltype(v16), const _Float16>::value, "");
-  static_assert(v16 == 1.F16, "");
-
-  static constexpr auto v17 = 1.e0F16; // OK.
-  static_assert(is_same<decltype(v17), const _Float16>::value, "");
-  static_assert(v17 == 1.F16, "");
 }
 
 void floating_point_complex_suffix() {

Modified: clang-tools-extra/branches/release_80/test/clang-tidy/readability-uppercase-literal-suffix-hexadecimal-floating-point.cpp
URL: http://llvm.org/viewvc/llvm-project/clang-tools-extra/branches/release_80/test/clang-tidy/readability-uppercase-literal-suffix-hexadecimal-floating-point.cpp?rev=352368&r1=352367&r2=352368&view=diff
==============================================================================
--- clang-tools-extra/branches/release_80/test/clang-tidy/readability-uppercase-literal-suffix-hexadecimal-floating-point.cpp (original)
+++ clang-tools-extra/branches/release_80/test/clang-tidy/readability-uppercase-literal-suffix-hexadecimal-floating-point.cpp Mon Jan 28 06:13:58 2019
@@ -93,21 +93,6 @@ void floating_point_suffix() {
   static constexpr auto v12 = 0xfp0Q; // OK.
   static_assert(is_same<decltype(v12), const __float128>::value, "");
   static_assert(v12 == 0xfp0, "");
-
-  // _Float16
-
-  static constexpr auto v13 = 0xfp0f16;
-  // CHECK-MESSAGES: :[[@LINE-1]]:31: warning: floating point literal has suffix 'f16', which is not uppercase
-  // CHECK-MESSAGES-NEXT: static constexpr auto v13 = 0xfp0f16;
-  // CHECK-MESSAGES-NEXT: ^    ~
-  // CHECK-MESSAGES-NEXT: {{^ *}}F16{{$}}
-  // CHECK-FIXES: static constexpr auto v13 = 0xfp0F16;
-  static_assert(is_same<decltype(v13), const _Float16>::value, "");
-  static_assert(v13 == 0xfp0F16, "");
-
-  static constexpr auto v14 = 0xfp0F16; // OK.
-  static_assert(is_same<decltype(v14), const _Float16>::value, "");
-  static_assert(v14 == 0xfp0F16, "");
 }
 
 void floating_point_complex_suffix() {