[debuginfo-tests] 470e82c - Revert "[Dexter] Add support for Windows to regression test suite."

Davide Italiano via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 27 13:12:34 PDT 2020


Author: Davide Italiano
Date: 2020-03-27T13:12:24-07:00
New Revision: 470e82cd06062a3fc3c142782b4d46d711b7c19c

URL: https://github.com/llvm/llvm-project/commit/470e82cd06062a3fc3c142782b4d46d711b7c19c
DIFF: https://github.com/llvm/llvm-project/commit/470e82cd06062a3fc3c142782b4d46d711b7c19c.diff

LOG: Revert "[Dexter] Add support for Windows to regression test suite."

This reverts commit 89025da9f676aebff7daf055824d6fd102a70c34 as
it breaks the lldb macOS bot.

Added: 
    

Modified: 
    debuginfo-tests/dexter/feature_tests/commands/penalty/expect_program_state.cpp
    debuginfo-tests/dexter/feature_tests/commands/penalty/expect_step_kinds.cpp
    debuginfo-tests/dexter/feature_tests/commands/penalty/expect_step_order.cpp
    debuginfo-tests/dexter/feature_tests/commands/penalty/expect_watch_type.cpp
    debuginfo-tests/dexter/feature_tests/commands/penalty/expect_watch_value.cpp
    debuginfo-tests/dexter/feature_tests/commands/penalty/unreachable.cpp
    debuginfo-tests/dexter/feature_tests/commands/perfect/expect_program_state.cpp
    debuginfo-tests/dexter/feature_tests/commands/perfect/expect_step_kind/direction.cpp
    debuginfo-tests/dexter/feature_tests/commands/perfect/expect_step_kind/func.cpp
    debuginfo-tests/dexter/feature_tests/commands/perfect/expect_step_kind/func_external.cpp
    debuginfo-tests/dexter/feature_tests/commands/perfect/expect_step_kind/recursive.cpp
    debuginfo-tests/dexter/feature_tests/commands/perfect/expect_step_kind/small_loop.cpp
    debuginfo-tests/dexter/feature_tests/commands/perfect/expect_step_order.cpp
    debuginfo-tests/dexter/feature_tests/commands/perfect/expect_watch_type.cpp
    debuginfo-tests/dexter/feature_tests/commands/perfect/expect_watch_value.cpp
    debuginfo-tests/dexter/feature_tests/commands/perfect/unreachable.cpp
    debuginfo-tests/dexter/feature_tests/subtools/view.cpp
    debuginfo-tests/lit.cfg.py

Removed: 
    


################################################################################
diff  --git a/debuginfo-tests/dexter/feature_tests/commands/penalty/expect_program_state.cpp b/debuginfo-tests/dexter/feature_tests/commands/penalty/expect_program_state.cpp
index 2a2355cc531e..476906846793 100644
--- a/debuginfo-tests/dexter/feature_tests/commands/penalty/expect_program_state.cpp
+++ b/debuginfo-tests/dexter/feature_tests/commands/penalty/expect_program_state.cpp
@@ -2,7 +2,11 @@
 //      Check that \DexExpectProgramState correctly applies a penalty when
 //      an expected program state is never found.
 //
-// RUN: not %dexter_regression_test -- %s | FileCheck %s
+// REQUIRES: system-linux, lldb
+//
+// RUN: not %dexter_base test --fail-lt 1.0 -w \
+// RUN:     --builder 'clang' --debugger 'lldb' --cflags "-O0 -glldb" -- %s \
+// RUN:     | FileCheck %s
 // CHECK: expect_program_state.cpp:
 
 int GCD(int lhs, int rhs)

diff  --git a/debuginfo-tests/dexter/feature_tests/commands/penalty/expect_step_kinds.cpp b/debuginfo-tests/dexter/feature_tests/commands/penalty/expect_step_kinds.cpp
index a65e7f5100cd..9612c068ad61 100644
--- a/debuginfo-tests/dexter/feature_tests/commands/penalty/expect_step_kinds.cpp
+++ b/debuginfo-tests/dexter/feature_tests/commands/penalty/expect_step_kinds.cpp
@@ -2,7 +2,11 @@
 //      Check that \DexExpectStepKind correctly applies a penalty when
 //      unexpected step kinds are encountered.
 //
-// RUN: not %dexter_regression_test -- %s | FileCheck %s
+// REQUIRES: system-linux, lldb
+//
+// RUN: not %dexter_base test --fail-lt 1.0 -w  \
+// RUN:     --builder 'clang' --debugger 'lldb' --cflags "-O0 -g" -- %s \
+// RUN:     | FileCheck %s
 // CHECK: expect_step_kinds.cpp:
 
 int abs(int i){

diff  --git a/debuginfo-tests/dexter/feature_tests/commands/penalty/expect_step_order.cpp b/debuginfo-tests/dexter/feature_tests/commands/penalty/expect_step_order.cpp
index b02dd0275714..e52a7fffe203 100644
--- a/debuginfo-tests/dexter/feature_tests/commands/penalty/expect_step_order.cpp
+++ b/debuginfo-tests/dexter/feature_tests/commands/penalty/expect_step_order.cpp
@@ -2,7 +2,11 @@
 //      Check that \DexExpectStepOrder correctly applies a penalty for steps
 //      found out of expected order.
 //
-// RUN: not %dexter_regression_test -- %s | FileCheck %s
+// REQUIRES: system-linux, lldb
+//
+// RUN: not %dexter_base test --fail-lt 1.0 -w \
+// RUN:     --builder 'clang' --debugger 'lldb' --cflags "-O0 -g" -- %s \
+// RUN:     | FileCheck %s
 // CHECK: expect_step_order.cpp:
 
 int main()

diff  --git a/debuginfo-tests/dexter/feature_tests/commands/penalty/expect_watch_type.cpp b/debuginfo-tests/dexter/feature_tests/commands/penalty/expect_watch_type.cpp
index c81de5f80069..145eeed7ef86 100644
--- a/debuginfo-tests/dexter/feature_tests/commands/penalty/expect_watch_type.cpp
+++ b/debuginfo-tests/dexter/feature_tests/commands/penalty/expect_watch_type.cpp
@@ -2,11 +2,11 @@
 //      Check that \DexExpectWatchType applies penalties when expected
 //      types are not found and unexpected types are.
 //
-// NOTE: This test passes but not in the expected way on Windows.
-// TODO: Reduce this test's coverage and be more specific about
-// expected behaviour.
+// REQUIRES: system-linux, lldb
 //
-// RUN: not %dexter_regression_test -- %s | FileCheck %s
+// RUN: not %dexter_base test --fail-lt 1.0 -w \
+// RUN:     --builder 'clang' --debugger 'lldb' --cflags "-O0 -g" -- %s \
+// RUN:     | FileCheck %s
 // CHECK: expect_watch_type.cpp:
 
 template<class T>

diff  --git a/debuginfo-tests/dexter/feature_tests/commands/penalty/expect_watch_value.cpp b/debuginfo-tests/dexter/feature_tests/commands/penalty/expect_watch_value.cpp
index cc7d2debb4d8..b4c60fa32179 100644
--- a/debuginfo-tests/dexter/feature_tests/commands/penalty/expect_watch_value.cpp
+++ b/debuginfo-tests/dexter/feature_tests/commands/penalty/expect_watch_value.cpp
@@ -2,7 +2,11 @@
 //      Check that \DexExpectWatchValue correctly applies a penalty when
 //      expected values are not found.
 //
-// RUN: not %dexter_regression_test -- %s | FileCheck %s
+// REQUIRES: system-linux, lldb
+//
+// RUN: not %dexter_base test --fail-lt 1.0 -w \
+// RUN:     --builder 'clang' --debugger 'lldb' --cflags "-O0 -g" -- %s \
+// RUN:     | FileCheck %s
 // CHECK: expect_watch_value.cpp:
 
 int main()

diff  --git a/debuginfo-tests/dexter/feature_tests/commands/penalty/unreachable.cpp b/debuginfo-tests/dexter/feature_tests/commands/penalty/unreachable.cpp
index 3d3e76c21dbf..5cea1c4257d0 100644
--- a/debuginfo-tests/dexter/feature_tests/commands/penalty/unreachable.cpp
+++ b/debuginfo-tests/dexter/feature_tests/commands/penalty/unreachable.cpp
@@ -2,7 +2,11 @@
 //      Check that \DexUnreachable correctly applies a penalty if the command
 //      line is stepped on.
 //
-// RUN: not %dexter_regression_test -- %s | FileCheck %s
+// REQUIRES: system-linux, lldb
+//
+// RUN: not %dexter_base test --fail-lt 1.0 -w \
+// RUN:     --builder 'clang' --debugger 'lldb' --cflags "-O0 -g" -- %s \
+// RUN:     | FileCheck %s
 // CHECK: unreachable.cpp:
 
 int

diff  --git a/debuginfo-tests/dexter/feature_tests/commands/perfect/expect_program_state.cpp b/debuginfo-tests/dexter/feature_tests/commands/perfect/expect_program_state.cpp
index 7137998684aa..ebb4766f089a 100644
--- a/debuginfo-tests/dexter/feature_tests/commands/perfect/expect_program_state.cpp
+++ b/debuginfo-tests/dexter/feature_tests/commands/perfect/expect_program_state.cpp
@@ -2,7 +2,11 @@
 //      Check that \DexExpectWatchValue applies no penalties when expected
 //      program states are found.
 //
-// RUN: %dexter_regression_test -- %s | FileCheck %s
+// REQUIRES: system-linux, lldb
+//
+// RUN: %dexter_base test --fail-lt 1.0 -w \
+// RUN:     --builder 'clang' --debugger 'lldb' --cflags "-O0 -glldb" -- %s \
+// RUN:     | FileCheck %s
 // CHECK: expect_program_state.cpp:
 
 int GCD(int lhs, int rhs)

diff  --git a/debuginfo-tests/dexter/feature_tests/commands/perfect/expect_step_kind/direction.cpp b/debuginfo-tests/dexter/feature_tests/commands/perfect/expect_step_kind/direction.cpp
index 6f4e85f925a6..3ff0e9fecbc6 100644
--- a/debuginfo-tests/dexter/feature_tests/commands/perfect/expect_step_kind/direction.cpp
+++ b/debuginfo-tests/dexter/feature_tests/commands/perfect/expect_step_kind/direction.cpp
@@ -5,10 +5,11 @@
 //      'VERTICAL_FORWARD' for every step onto a greater source line number in
 //      the same function.
 //
-// TODO: The dbgeng debugger does not support column step reporting at present.
-// XFAIL: system-windows
+// REQUIRES: system-linux, lldb
 //
-// RUN: %dexter_regression_test -- %s | FileCheck %s
+// RUN: %dexter_base test --fail-lt 1.0 -w  \
+// RUN:     --builder 'clang' --debugger 'lldb' --cflags "-O0 -g" -- %s \
+// RUN:     | FileCheck %s
 // CHECK: direction.cpp:
 
 int func(int i) {

diff  --git a/debuginfo-tests/dexter/feature_tests/commands/perfect/expect_step_kind/func.cpp b/debuginfo-tests/dexter/feature_tests/commands/perfect/expect_step_kind/func.cpp
index c845c4d2bc5e..d5ab3ffbd036 100644
--- a/debuginfo-tests/dexter/feature_tests/commands/perfect/expect_step_kind/func.cpp
+++ b/debuginfo-tests/dexter/feature_tests/commands/perfect/expect_step_kind/func.cpp
@@ -3,7 +3,11 @@
 //      trivial test. Expect one 'FUNC' per call to a function which is defined
 //      in one of the source files in the test directory.
 //
-// RUN: %dexter_regression_test -- %s | FileCheck %s
+// REQUIRES: system-linux, lldb
+//
+// RUN: %dexter_base test --fail-lt 1.0 -w  \
+// RUN:     --builder 'clang' --debugger 'lldb' --cflags "-O0 -g" -- %s \
+// RUN:     | FileCheck %s
 // CHECK: func.cpp:
 
 int func(int i) {

diff  --git a/debuginfo-tests/dexter/feature_tests/commands/perfect/expect_step_kind/func_external.cpp b/debuginfo-tests/dexter/feature_tests/commands/perfect/expect_step_kind/func_external.cpp
index b496b3b5a909..6dff5c802968 100644
--- a/debuginfo-tests/dexter/feature_tests/commands/perfect/expect_step_kind/func_external.cpp
+++ b/debuginfo-tests/dexter/feature_tests/commands/perfect/expect_step_kind/func_external.cpp
@@ -2,11 +2,13 @@
 //      Check that \DexExpectStepKind correctly counts 'FUNC_EXTERNAL' steps
 //      for a trivial test. Expect one 'FUNC_EXTERNAL' per external call.
 //
-// XFAIL:*
-// This fails right now on my linux and windows machine, needs examining as to
-// why.
+// REQUIRES: system-linux, lldb
+// XFAIL: system-linux
+// This fails right now on my linux machine, needs examining as to why.
 //
-// RUN: %dexter_regression_test -- %s | FileCheck %s
+// RUN: %dexter --fail-lt 1.0 -w  \
+// RUN:     --builder 'clang' --debugger 'lldb' --cflags "-O0 -g" -- %s \
+// RUN:     | FileCheck %s
 // CHECK: func_external.cpp:
 
 #include <cstdlib>

diff  --git a/debuginfo-tests/dexter/feature_tests/commands/perfect/expect_step_kind/recursive.cpp b/debuginfo-tests/dexter/feature_tests/commands/perfect/expect_step_kind/recursive.cpp
index 31dc6c66f931..f2b54e442679 100644
--- a/debuginfo-tests/dexter/feature_tests/commands/perfect/expect_step_kind/recursive.cpp
+++ b/debuginfo-tests/dexter/feature_tests/commands/perfect/expect_step_kind/recursive.cpp
@@ -3,7 +3,11 @@
 //      Specifically, ensure recursive calls count towards 'FUNC' and not
 //      'VERTICAL_BACKWARD'.
 //
-// RUN: %dexter_regression_test -- %s | FileCheck %s
+// REQUIRES: system-linux, lldb
+//
+// RUN: %dexter_base test --fail-lt 1.0 -w  \
+// RUN:     --builder 'clang' --debugger 'lldb' --cflags "-O0 -g" -- %s \
+// RUN:     | FileCheck %s
 // CHECK: recursive.cpp:
 
 int func(int i) {

diff  --git a/debuginfo-tests/dexter/feature_tests/commands/perfect/expect_step_kind/small_loop.cpp b/debuginfo-tests/dexter/feature_tests/commands/perfect/expect_step_kind/small_loop.cpp
index 63bb197578ad..1f9060747ec2 100644
--- a/debuginfo-tests/dexter/feature_tests/commands/perfect/expect_step_kind/small_loop.cpp
+++ b/debuginfo-tests/dexter/feature_tests/commands/perfect/expect_step_kind/small_loop.cpp
@@ -3,7 +3,11 @@
 //      where the last source line in the loop is a call. Expect steps out
 //      of a function to a line before the call to count as 'VERTICAL_BACKWARD'.
 //
-// RUN: %dexter_regression_test -- %s | FileCheck %s
+// REQUIRES: system-linux, lldb
+//
+// RUN: %dexter_base test --fail-lt 1.0 -w  \
+// RUN:     --builder 'clang' --debugger 'lldb' --cflags "-O0 -g" -- %s \
+// RUN:     | FileCheck %s
 // CHECK: small_loop.cpp:
 
 int func(int i){

diff  --git a/debuginfo-tests/dexter/feature_tests/commands/perfect/expect_step_order.cpp b/debuginfo-tests/dexter/feature_tests/commands/perfect/expect_step_order.cpp
index e24d771957ef..a89a0cee3cc5 100644
--- a/debuginfo-tests/dexter/feature_tests/commands/perfect/expect_step_order.cpp
+++ b/debuginfo-tests/dexter/feature_tests/commands/perfect/expect_step_order.cpp
@@ -2,7 +2,11 @@
 //      Check that \DexExpectStepOrder applies no penalty when the expected
 //      order is found.
 //
-// RUN: %dexter_regression_test -- %s | FileCheck %s
+// REQUIRES: system-linux, lldb
+//
+// RUN: %dexter_base test --fail-lt 1.0 -w \
+// RUN:     --builder 'clang' --debugger 'lldb' --cflags "-O0 -g" -- %s \
+// RUN:     | FileCheck %s
 // CHECK: expect_step_order.cpp:
 
 int main()

diff  --git a/debuginfo-tests/dexter/feature_tests/commands/perfect/expect_watch_type.cpp b/debuginfo-tests/dexter/feature_tests/commands/perfect/expect_watch_type.cpp
index d3ef79bbb7e7..41593837a9c8 100644
--- a/debuginfo-tests/dexter/feature_tests/commands/perfect/expect_watch_type.cpp
+++ b/debuginfo-tests/dexter/feature_tests/commands/perfect/expect_watch_type.cpp
@@ -2,11 +2,11 @@
 //      Check that \DexExpectWatchType applies no penalties when expected
 //      types are found.
 //
-// TODO: On Windows WITH dbgeng, This test takes a long time to run and doesn't evaluate type values
-// in the same manner as LLDB.
-// XFAIL: system-windows
+// REQUIRES: system-linux, lldb
 //
-// RUN: %dexter_regression_test -- %s | FileCheck %s
+// RUN: %dexter_base test --fail-lt 1.0 -w \
+// RUN:     --builder 'clang' --debugger 'lldb' --cflags "-O0 -g" -- %s \
+// RUN:     | FileCheck %s
 // CHECK: expect_watch_type.cpp:
 
 template<class T>

diff  --git a/debuginfo-tests/dexter/feature_tests/commands/perfect/expect_watch_value.cpp b/debuginfo-tests/dexter/feature_tests/commands/perfect/expect_watch_value.cpp
index b86cc3187132..035e41136980 100644
--- a/debuginfo-tests/dexter/feature_tests/commands/perfect/expect_watch_value.cpp
+++ b/debuginfo-tests/dexter/feature_tests/commands/perfect/expect_watch_value.cpp
@@ -2,7 +2,11 @@
 //      Check that \DexExpectWatchValue applies no penalties when expected
 //      values are found.
 //
-// RUN: %dexter_regression_test -- %s | FileCheck %s
+// REQUIRES: system-linux, lldb
+//
+// RUN: %dexter_base test --fail-lt 1.0 -w \
+// RUN:     --builder 'clang' --debugger 'lldb' --cflags "-O0 -g" -- %s \
+// RUN:     | FileCheck %s
 // CHECK: expect_watch_value.cpp:
 
 unsigned long Factorial(int n) {

diff  --git a/debuginfo-tests/dexter/feature_tests/commands/perfect/unreachable.cpp b/debuginfo-tests/dexter/feature_tests/commands/perfect/unreachable.cpp
index cf889f81f2fe..b158b4851be4 100644
--- a/debuginfo-tests/dexter/feature_tests/commands/perfect/unreachable.cpp
+++ b/debuginfo-tests/dexter/feature_tests/commands/perfect/unreachable.cpp
@@ -2,7 +2,11 @@
 //    Check that \DexUnreachable has no effect if the command line is never
 //    stepped on.
 //
-// RUN: %dexter_regression_test -- %s | FileCheck %s
+// REQUIRES: system-linux, lldb
+//
+// RUN: %dexter_base test --fail-lt 1.0 -w \
+// RUN:     --builder 'clang' --debugger 'lldb' --cflags "-O0 -g" -- %s \
+// RUN:     | FileCheck %s
 // CHECK: unreachable.cpp:
 
 int main()

diff  --git a/debuginfo-tests/dexter/feature_tests/subtools/view.cpp b/debuginfo-tests/dexter/feature_tests/subtools/view.cpp
index 03e0b9420fb1..43598dbdc9b9 100644
--- a/debuginfo-tests/dexter/feature_tests/subtools/view.cpp
+++ b/debuginfo-tests/dexter/feature_tests/subtools/view.cpp
@@ -1,7 +1,11 @@
 // Purpose:
 //      Check the `view` subtool works with typical inputs.
 //
-// RUN: %dexter_regression_test --results %t -- %s
+// REQUIRES: system-linux, lldb
+//
+// RUN: %dexter_base test --fail-lt 1.0 -w \
+// RUN:    --builder 'clang' --debugger 'lldb' --cflags "-O0 -g" \
+// RUN:    --results %t -- %s
 //
 // RUN: %dexter_base view %t/view.cpp.dextIR | FileCheck %s
 // CHECK: ## BEGIN

diff  --git a/debuginfo-tests/lit.cfg.py b/debuginfo-tests/lit.cfg.py
index fb2f72357b26..2d93b8da9bf6 100644
--- a/debuginfo-tests/lit.cfg.py
+++ b/debuginfo-tests/lit.cfg.py
@@ -112,35 +112,6 @@ def get_required_attr(config, attr_name):
 dexter_base_cmd = '"{}" "{}"'.format(config.python3_executable, dexter_path)
 tools.append(ToolSubst('%dexter_base', dexter_base_cmd))
 
-# Set up commands for DexTer regression tests.
-# Builder, debugger, optimisation level and several other flags 
diff er
-# depending on whether we're running a unix like or windows os.
-if platform.system() == 'Windows': 
-  dexter_regression_test_builder = '--builder clang-cl_vs2015'
-  dexter_regression_test_debugger = '--debugger dbgeng'
-  dexter_regression_test_cflags = '--cflags "/Zi /Od"'
-  dexter_regression_test_ldflags = '--ldflags "/Zi"'
-else:
-  dexter_regression_test_builder = '--builder clang'
-  dexter_regression_test_debugger = "--debugger lldb"
-  dexter_regression_test_cflags = '--cflags "-O0 -glldb"'
-  dexter_regression_test_ldflags = ''
-
-# Typical command would take the form:
-# ./path_to_py/python.exe ./path_to_dex/dexter.py test --fail-lt 1.0 -w --builder clang --debugger lldb --cflags '-O0 -g'
-dexter_regression_test_command = ' '.join(
-  # "python3", "dexter.py", test, fail_mode, builder, debugger, cflags, ldflags
-  ["{}".format(config.python3_executable),
-  "{}".format(dexter_path),
-  'test',
-  '--fail-lt 1.0 -w',
-  dexter_regression_test_builder,
-  dexter_regression_test_debugger,
-  dexter_regression_test_cflags,
-  dexter_regression_test_ldflags])
-
-tools.append(ToolSubst('%dexter_regression_test', dexter_regression_test_command))
-
 tool_dirs = [config.llvm_tools_dir]
 
 llvm_config.add_tool_substitutions(tools, tool_dirs)


        


More information about the llvm-commits mailing list