[PATCH] D60749: [Test] Remove obsolete test.
Michael Kruse via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Apr 15 20:24:04 PDT 2019
Meinersbur created this revision.
Meinersbur added reviewers: hfinkel, aaron.ballman, tyler.nowicki, TylerNowicki.
Meinersbur added a project: clang.
The FIXME of this test case has been addressed in r335084/r338800. Its execution still does not succeed because of multiple syntax errors.
First, the "clang" namespace is missing on each of the 4 pragmas. Second, the pragma for defining the vector width is "vectorize_width(4)" instead of "vectorize(4)". Third, the pragma for defining the interleave factor is "interleave_count(8)" instead of "interleave(8)".
The file was already using the wrong syntax when added in r210925 2014-06-13. The file ast-print-pragmas.cpp already checks for the correct pragma order, making this test redundant even if fixed.
Repository:
rC Clang
https://reviews.llvm.org/D60749
Files:
test/AST/ast-print-pragmas-xfail.cpp
Index: test/AST/ast-print-pragmas-xfail.cpp
===================================================================
--- test/AST/ast-print-pragmas-xfail.cpp
+++ /dev/null
@@ -1,21 +0,0 @@
-// RUN: %clang_cc1 %s -ast-print -o - | FileCheck %s
-
-// FIXME: Test fails because attribute order is reversed by ParsedAttributes.
-// XFAIL: *
-
-void run1(int *List, int Length) {
- int i = 0;
-// CHECK: #pragma loop vectorize(4)
-// CHECK-NEXT: #pragma loop interleave(8)
-// CHECK-NEXT: #pragma loop vectorize(enable)
-// CHECK-NEXT: #pragma loop interleave(enable)
-#pragma loop vectorize(4)
-#pragma loop interleave(8)
-#pragma loop vectorize(enable)
-#pragma loop interleave(enable)
-// CHECK-NEXT: while (i < Length)
- while (i < Length) {
- List[i] = i;
- i++;
- }
-}
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D60749.195293.patch
Type: text/x-patch
Size: 774 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20190416/fa098fb6/attachment.bin>
More information about the cfe-commits
mailing list