Skip to content

Commit ea9b7c0

Browse files
committed
Synchronise goto-cc command-line options to re-enable Windows regression tests
object-bits was not supported by all of goto-cc's command-line front-end variants. While at it, also make other options consistently available.
1 parent 16ed35a commit ea9b7c0

File tree

7 files changed

+15
-7
lines changed

7 files changed

+15
-7
lines changed
+1-3
Original file line numberDiff line numberDiff line change
@@ -1,11 +1,9 @@
11
if(WIN32)
22
set(is_windows true)
3-
set(exclude_broken_windows_tests -X winbug)
43
else()
54
set(is_windows false)
6-
set(exclude_broken_windows_tests "")
75
endif()
86

97
add_test_pl_tests(
10-
"${CMAKE_CURRENT_SOURCE_DIR}/chain.sh $<TARGET_FILE:goto-cc> $<TARGET_FILE:cbmc> ${is_windows}" ${exclude_broken_windows_tests}
8+
"${CMAKE_CURRENT_SOURCE_DIR}/chain.sh $<TARGET_FILE:goto-cc> $<TARGET_FILE:cbmc> ${is_windows}"
119
)

Diff for: regression/goto-cc-cbmc-shared-options/object-bits/fewer_bits.desc

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
CORE winbug
1+
CORE
22
test.c
33
--function main --object-bits 6
44
^EXIT=10$

Diff for: regression/goto-cc-cbmc-shared-options/object-bits/more_bits.desc

+1-1
Original file line numberDiff line numberDiff line change
@@ -1,4 +1,4 @@
1-
CORE winbug
1+
CORE
22
test.c
33
--function main --object-bits 10
44
^EXIT=10$

Diff for: src/goto-cc/armcc_cmdline.cpp

+6
Original file line numberDiff line numberDiff line change
@@ -44,6 +44,9 @@ static const char *options_no_arg[]=
4444
"--no-arch",
4545
"--no-library",
4646
"--string-abstraction",
47+
"--validate-goto-model",
48+
"-?",
49+
"--export-file-local-symbols",
4750

4851
// armcc
4952
"--help",
@@ -256,6 +259,9 @@ static const std::vector<std::string> options_with_arg
256259
// goto-cc specific
257260
"--verbosity",
258261
"--function",
262+
"--print-rejected-preprocessed-source",
263+
"--mangle-suffix",
264+
"--object-bits",
259265

260266
// armcc-specific
261267
"-D",

Diff for: src/goto-cc/bcc_cmdline.cpp

+2
Original file line numberDiff line numberDiff line change
@@ -24,6 +24,8 @@ const char *goto_bcc_options_with_argument[]=
2424
"--native-compiler",
2525
"--native-linker",
2626
"--print-rejected-preprocessed-source",
27+
"--mangle-suffix",
28+
"--object-bits",
2729
nullptr
2830
};
2931

Diff for: src/goto-cc/ms_cl_cmdline.cpp

+2-1
Original file line numberDiff line numberDiff line change
@@ -48,6 +48,7 @@ const char *non_ms_cl_options[]=
4848
"--validate-goto-model",
4949
"--export-file-local-symbols",
5050
"--mangle-suffix",
51+
"--object-bits",
5152
nullptr
5253
};
5354
// clang-format on
@@ -63,7 +64,7 @@ bool ms_cl_cmdlinet::parse(const std::vector<std::string> &arguments)
6364

6465
if(
6566
arguments[i] == "--verbosity" || arguments[i] == "--function" ||
66-
arguments[i] == "--mangle-suffix")
67+
arguments[i] == "--mangle-suffix" || arguments[i] == "--object-bits")
6768
{
6869
if(i < arguments.size() - 1)
6970
{

Diff for: src/goto-cc/ms_link_cmdline.cpp

+2-1
Original file line numberDiff line numberDiff line change
@@ -25,7 +25,8 @@ Author: Daniel Kroening
2525
const char *non_ms_link_options[]=
2626
{
2727
"--help",
28-
"--verbosity"
28+
"--verbosity",
29+
"--validate-goto-model"
2930
};
3031

3132
bool ms_link_cmdlinet::parse(const std::vector<std::string> &arguments)

0 commit comments

Comments
 (0)