]> granicus.if.org Git - clang/commit
[OpenMP] Update CHECK-DAG usage in target_parallel_codegen.cpp
authorJoel E. Denny <jdenny.ornl@gmail.com>
Tue, 20 Nov 2018 22:05:23 +0000 (22:05 +0000)
committerJoel E. Denny <jdenny.ornl@gmail.com>
Tue, 20 Nov 2018 22:05:23 +0000 (22:05 +0000)
commit3391bf6a2d48276c99ac52101ca0845cba29f2c4
treeb5fd07999474c82ce569c138fdc8300f4326a8bd
parent32141ec751dcd24836840bf5da033aaa169c71b1
[OpenMP] Update CHECK-DAG usage in target_parallel_codegen.cpp

This patch adjusts a test not to depend on deprecated FileCheck
behavior that permits overlapping matches within a block of CHECK-DAG
directives.  Thus, this patch also removes uses of FileCheck's
-allow-deprecated-dag-overlap command-line option.

There were two issues in this test:

1. There were sets of patterns for store instructions in which a
pattern X could match a superset of a pattern Y.  While X appeared
before Y, Y's intended match appeared before X's intended match.  The
result was that X matched Y's intended match.  Under the old
overlapping behavior, Y also matched Y's intended match.  Under the
new non-overlapping behavior, Y had nothing left to match.  This patch
fixes this by gathering these sets in one place and putting the most
specific patterns (Y) before the more general patterns (X).

2. The CHECK-DAG patterns involving the variables CBPADDR3 and
CBPADDR4 were the same, but there was only one match in the text, so
CBPADDR4 patterns had nothing to match under the new non-overlapping
behavior.  Moreover, a preceding related series of directives had
variables (SADDR0, BPADDR0, etc.) numbered only 0 through 4, but this
series had variables numbered 0 through 5.  Assuming CBPADDR4's
directives were not intended, this patch removes them.

Reviewed By: ABataev

Differential Revision: https://reviews.llvm.org/D54765

git-svn-id: https://llvm.org/svn/llvm-project/cfe/trunk@347351 91177308-0d34-0410-b5e6-96231b3b80d8
test/OpenMP/target_parallel_codegen.cpp