Skip to content
New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Test failure on msp430-elfbare target #1713

Closed
CohenArthur opened this issue Jan 2, 2023 · 4 comments
Closed

Test failure on msp430-elfbare target #1713

CohenArthur opened this issue Jan 2, 2023 · 4 comments
Assignees
Labels

Comments

@CohenArthur
Copy link
Member

CohenArthur commented Jan 2, 2023

Rust related issues
=====================

 --target=msp430-elfbare
~~~~~~~~~~~~~~~~~~~~~~~~~
	/var/lib/laminar/run/gcc-msp430-elfbare/24/toolchain-build/./gcc/xgcc -B/var/lib/laminar/run/gcc-msp430-elfbare/24/toolchain-build/./gcc/  -xrust -frust-incomplete-and-experimental-compiler-do-not-use -nostdinc /dev/null -S -o /dev/null -fself-test=../../gcc/gcc/testsuite/selftests
	<built-in>: internal compiler error: Segmentation fault
	0xf2efbf crash_signal
	      ../../gcc/gcc/toplev.cc:314
	0x120c8c7 build_function_type(tree_node*, tree_node*, bool)
	      ../../gcc/gcc/tree.cc:7360
	0x120cc20 build_function_type_list(tree_node*, ...)
	      ../../gcc/gcc/tree.cc:7442
	0x120d16b build_common_builtin_nodes()
	      ../../gcc/gcc/tree.cc:9883
	0x8449b4 grs_langhook_init
	      ../../gcc/gcc/rust/rust-lang.cc:132
	0x8427b2 lang_dependent_init
	      ../../gcc/gcc/toplev.cc:1815
	0x8427b2 do_compile
	      ../../gcc/gcc/toplev.cc:2110
	Please submit a full bug report, with preprocessed source (by using -freport-bug).
	Please include the complete backtrace with any bug report.
	See [<https://gcc.gnu.org/bugs/>](https://gcc.gnu.org/bugs/) for instructions.
	make[1]: *** [../../gcc/gcc/rust/Make-lang.in:275: s-selftest-rust] Error 1

From https://gcc.gnu.org/pipermail/gcc-patches/2022-December/608802.html

@CohenArthur CohenArthur added the bug label Jan 2, 2023
@CohenArthur CohenArthur added this to the Final upstream patches milestone Jan 2, 2023
@tschwinge tschwinge self-assigned this Feb 22, 2023
@tschwinge
Copy link
Member

Patch posted: https://inbox.sourceware.org/gcc-patches/871qmi6iya.fsf@euler.schwinge.homeip.net "Rust: In 'type_for_mode' langhook also consider all 'int_n' modes/types".

wangliu-iscas pushed a commit to plctlab/patchwork-gcc that referenced this issue Feb 22, 2023
As reported in "Rust related issues" at the end of
<https://inbox.sourceware.org/gcc-patches/20221219212344.ewtuzd2jtztm77x2@lug-owl.de>
"Modula-2 / Rust: Many targets failing",
<Rust-GCC/gccrs#1713>
"Test failure on msp430-elfbare target", for '--target=msp430-elfbar' we ICE:

    <built-in>: internal compiler error: Segmentation fault
    0xf2efbf crash_signal
          ../../gcc/gcc/toplev.cc:314
    0x120c8c7 build_function_type(tree_node*, tree_node*, bool)
          ../../gcc/gcc/tree.cc:7360
    0x120cc20 build_function_type_list(tree_node*, ...)
          ../../gcc/gcc/tree.cc:7442
    0x120d16b build_common_builtin_nodes()
          ../../gcc/gcc/tree.cc:9883
    0x8449b4 grs_langhook_init
          ../../gcc/gcc/rust/rust-lang.cc:132
    [...]

This is due to 'tmp == NULL' for 'gcc/tree.cc:build_common_builtin_nodes':

    tmp = lang_hooks.types.type_for_mode (targetm.eh_return_filter_mode (), 0);

..., were 'targetm.eh_return_filter_mode' (that is,
'gcc/targhooks.cc:default_eh_return_filter_mode',
via calling 'targetm.unwind_word_mode':
'gcc/config/msp430/msp430.cc:msp430_unwind_word_mode') returns 'PSImode',
and 'lang_hooks.types.type_for_mode' (that is,
'gcc/rust/rust-lang.cc:grs_langhook_type_for_mode') returns 'NULL' for
'PSImode'.

As, for example, discussed in <https://gcc.gnu.org/PR46805>
"ICE: SIGSEGV in optab_for_tree_code (optabs.c:407) with -O -fno-tree-scev-cprop -ftree-vectorize",
we have to support "random" modes/types in the 'type_for_mode' langhook.

	gcc/rust/
	* rust-lang.cc (grs_langhook_type_for_mode): Also consider all
	'int_n' modes/types.
tschwinge added a commit to tschwinge/gcc that referenced this issue Feb 22, 2023
As reported in "Rust related issues" at the end of
<https://inbox.sourceware.org/gcc-patches/20221219212344.ewtuzd2jtztm77x2@lug-owl.de>
"Modula-2 / Rust: Many targets failing",
<Rust-GCC/gccrs#1713>
"Test failure on msp430-elfbare target", for '--target=msp430-elfbar' we ICE:

    <built-in>: internal compiler error: Segmentation fault
    0xf2efbf crash_signal
          ../../gcc/gcc/toplev.cc:314
    0x120c8c7 build_function_type(tree_node*, tree_node*, bool)
          ../../gcc/gcc/tree.cc:7360
    0x120cc20 build_function_type_list(tree_node*, ...)
          ../../gcc/gcc/tree.cc:7442
    0x120d16b build_common_builtin_nodes()
          ../../gcc/gcc/tree.cc:9883
    0x8449b4 grs_langhook_init
          ../../gcc/gcc/rust/rust-lang.cc:132
    [...]

This is due to 'tmp == NULL' for 'gcc/tree.cc:build_common_builtin_nodes':

    tmp = lang_hooks.types.type_for_mode (targetm.eh_return_filter_mode (), 0);

..., were 'targetm.eh_return_filter_mode' (that is,
'gcc/targhooks.cc:default_eh_return_filter_mode',
via calling 'targetm.unwind_word_mode':
'gcc/config/msp430/msp430.cc:msp430_unwind_word_mode') returns 'PSImode',
and 'lang_hooks.types.type_for_mode' (that is,
'gcc/rust/rust-lang.cc:grs_langhook_type_for_mode') returns 'NULL' for
'PSImode'.

As, for example, discussed in <https://gcc.gnu.org/PR46805>
"ICE: SIGSEGV in optab_for_tree_code (optabs.c:407) with -O -fno-tree-scev-cprop -ftree-vectorize",
we have to support "random" modes/types in the 'type_for_mode' langhook.

	gcc/rust/
	* rust-lang.cc (grs_langhook_type_for_mode): Also consider all
	'int_n' modes/types.
@jbglaw
Copy link
Contributor

jbglaw commented Feb 25, 2023

Just had a successful build with upstream GCC: http://toolchain.lug-owl.de/laminar/jobs/gcc-msp430-elf/65

@CohenArthur
Copy link
Member Author

Nice! Thanks for confirming @jbglaw :) closing this

ZacWalk pushed a commit to Windows-on-ARM-Experiments/gcc-woarm64 that referenced this issue Mar 20, 2023
As reported in "Rust related issues" at the end of
<https://inbox.sourceware.org/gcc-patches/20221219212344.ewtuzd2jtztm77x2@lug-owl.de>
"Modula-2 / Rust: Many targets failing",
<Rust-GCC/gccrs#1713>
"Test failure on msp430-elfbare target", for '--target=msp430-elfbar' we ICE:

    <built-in>: internal compiler error: Segmentation fault
    0xf2efbf crash_signal
          ../../gcc/gcc/toplev.cc:314
    0x120c8c7 build_function_type(tree_node*, tree_node*, bool)
          ../../gcc/gcc/tree.cc:7360
    0x120cc20 build_function_type_list(tree_node*, ...)
          ../../gcc/gcc/tree.cc:7442
    0x120d16b build_common_builtin_nodes()
          ../../gcc/gcc/tree.cc:9883
    0x8449b4 grs_langhook_init
          ../../gcc/gcc/rust/rust-lang.cc:132
    [...]

This is due to 'tmp == NULL' for 'gcc/tree.cc:build_common_builtin_nodes':

    tmp = lang_hooks.types.type_for_mode (targetm.eh_return_filter_mode (), 0);

..., were 'targetm.eh_return_filter_mode' (that is,
'gcc/targhooks.cc:default_eh_return_filter_mode',
via calling 'targetm.unwind_word_mode':
'gcc/config/msp430/msp430.cc:msp430_unwind_word_mode') returns 'PSImode',
and 'lang_hooks.types.type_for_mode' (that is,
'gcc/rust/rust-lang.cc:grs_langhook_type_for_mode') returns 'NULL' for
'PSImode'.

As, for example, discussed in <https://gcc.gnu.org/PR46805>
"ICE: SIGSEGV in optab_for_tree_code (optabs.c:407) with -O -fno-tree-scev-cprop -ftree-vectorize",
we have to support "random" modes/types in the 'type_for_mode' langhook.

	gcc/rust/
	* rust-lang.cc (grs_langhook_type_for_mode): Also consider all
	'int_n' modes/types.
tschwinge added a commit that referenced this issue Apr 11, 2023
Yesterday's commit 3757e8d
"Merge commit '6baa95c9c5b3fea96fd22d03d961db4e4cf48d88' into HEAD [#2112]"
brought in the #1713 change from GCC upstream master branch that broke the
rules (via code copied verbatim from elsehwhere in GCC).

	gcc/rust/
	* rust-lang.cc (grs_langhook_type_for_mode): Placate clang-format.
tschwinge added a commit that referenced this issue Apr 11, 2023
Yesterday's commit 3757e8d
"Merge commit '6baa95c9c5b3fea96fd22d03d961db4e4cf48d88' into HEAD [#2112]"
brought in the #1713 change from GCC upstream master branch that broke the
rules (via code copied verbatim from elsehwhere in GCC).

	gcc/rust/
	* rust-lang.cc (grs_langhook_type_for_mode): Placate clang-format.
@jbglaw
Copy link
Contributor

jbglaw commented Apr 13, 2023

...and successful builds from the Rust-GCC repo also showed up.

CohenArthur pushed a commit to CohenArthur/gccrs that referenced this issue Nov 15, 2023
Yesterday's commit 3757e8d
"Merge commit '6baa95c9c5b3fea96fd22d03d961db4e4cf48d88' into HEAD [Rust-GCC#2112]"
brought in the Rust-GCC#1713 change from GCC upstream master branch that broke the
rules (via code copied verbatim from elsehwhere in GCC).

	gcc/rust/
	* rust-lang.cc (grs_langhook_type_for_mode): Placate clang-format.
CohenArthur pushed a commit to CohenArthur/gccrs that referenced this issue Nov 21, 2023
Yesterday's commit 3757e8d
"Merge commit '6baa95c9c5b3fea96fd22d03d961db4e4cf48d88' into HEAD [Rust-GCC#2112]"
brought in the Rust-GCC#1713 change from GCC upstream master branch that broke the
rules (via code copied verbatim from elsehwhere in GCC).

	gcc/rust/
	* rust-lang.cc (grs_langhook_type_for_mode): Placate clang-format.
CohenArthur pushed a commit to CohenArthur/gccrs that referenced this issue Nov 21, 2023
Yesterday's commit 3757e8d
"Merge commit '6baa95c9c5b3fea96fd22d03d961db4e4cf48d88' into HEAD [Rust-GCC#2112]"
brought in the Rust-GCC#1713 change from GCC upstream master branch that broke the
rules (via code copied verbatim from elsehwhere in GCC).

	gcc/rust/
	* rust-lang.cc (grs_langhook_type_for_mode): Placate clang-format.
CohenArthur pushed a commit to CohenArthur/gccrs that referenced this issue Jan 5, 2024
Yesterday's commit 3757e8d
"Merge commit '6baa95c9c5b3fea96fd22d03d961db4e4cf48d88' into HEAD [Rust-GCC#2112]"
brought in the Rust-GCC#1713 change from GCC upstream master branch that broke the
rules (via code copied verbatim from elsehwhere in GCC).

	gcc/rust/
	* rust-lang.cc (grs_langhook_type_for_mode): Placate clang-format.
CohenArthur pushed a commit to CohenArthur/gccrs that referenced this issue Jan 8, 2024
Yesterday's commit 3757e8d
"Merge commit '6baa95c9c5b3fea96fd22d03d961db4e4cf48d88' into HEAD [Rust-GCC#2112]"
brought in the Rust-GCC#1713 change from GCC upstream master branch that broke the
rules (via code copied verbatim from elsehwhere in GCC).

	gcc/rust/
	* rust-lang.cc (grs_langhook_type_for_mode): Placate clang-format.
CohenArthur pushed a commit to CohenArthur/gccrs that referenced this issue Jan 9, 2024
Yesterday's commit 3757e8d
"Merge commit '6baa95c9c5b3fea96fd22d03d961db4e4cf48d88' into HEAD [Rust-GCC#2112]"
brought in the Rust-GCC#1713 change from GCC upstream master branch that broke the
rules (via code copied verbatim from elsehwhere in GCC).

	gcc/rust/
	* rust-lang.cc (grs_langhook_type_for_mode): Placate clang-format.
CohenArthur pushed a commit to CohenArthur/gccrs that referenced this issue Jan 9, 2024
Yesterday's commit 3757e8d
"Merge commit '6baa95c9c5b3fea96fd22d03d961db4e4cf48d88' into HEAD [Rust-GCC#2112]"
brought in the Rust-GCC#1713 change from GCC upstream master branch that broke the
rules (via code copied verbatim from elsehwhere in GCC).

	gcc/rust/
	* rust-lang.cc (grs_langhook_type_for_mode): Placate clang-format.
CohenArthur pushed a commit to CohenArthur/gccrs that referenced this issue Jan 9, 2024
Yesterday's commit 3757e8d
"Merge commit '6baa95c9c5b3fea96fd22d03d961db4e4cf48d88' into HEAD [Rust-GCC#2112]"
brought in the Rust-GCC#1713 change from GCC upstream master branch that broke the
rules (via code copied verbatim from elsehwhere in GCC).

	gcc/rust/
	* rust-lang.cc (grs_langhook_type_for_mode): Placate clang-format.
CohenArthur pushed a commit to CohenArthur/gccrs that referenced this issue Jan 9, 2024
Yesterday's commit 3757e8d
"Merge commit '6baa95c9c5b3fea96fd22d03d961db4e4cf48d88' into HEAD [Rust-GCC#2112]"
brought in the Rust-GCC#1713 change from GCC upstream master branch that broke the
rules (via code copied verbatim from elsehwhere in GCC).

	gcc/rust/
	* rust-lang.cc (grs_langhook_type_for_mode): Placate clang-format.
CohenArthur pushed a commit to CohenArthur/gccrs that referenced this issue Jan 9, 2024
Yesterday's commit 3757e8d
"Merge commit '6baa95c9c5b3fea96fd22d03d961db4e4cf48d88' into HEAD [Rust-GCC#2112]"
brought in the Rust-GCC#1713 change from GCC upstream master branch that broke the
rules (via code copied verbatim from elsehwhere in GCC).

	gcc/rust/
	* rust-lang.cc (grs_langhook_type_for_mode): Placate clang-format.
CohenArthur pushed a commit to CohenArthur/gccrs that referenced this issue Jan 9, 2024
…_for_mode'

Yesterday's commit 3757e8d
"Merge commit '6baa95c9c5b3fea96fd22d03d961db4e4cf48d88' into HEAD [Rust-GCC#2112]"
brought in the Rust-GCC#1713 change from GCC upstream master branch that broke the
rules (via code copied verbatim from elsehwhere in GCC).

	gcc/rust/
	* rust-lang.cc (grs_langhook_type_for_mode): Placate clang-format.
CohenArthur pushed a commit to CohenArthur/gccrs that referenced this issue Jan 11, 2024
…_for_mode'

Yesterday's commit 3757e8d
"Merge commit '6baa95c9c5b3fea96fd22d03d961db4e4cf48d88' into HEAD [Rust-GCC#2112]"
brought in the Rust-GCC#1713 change from GCC upstream master branch that broke the
rules (via code copied verbatim from elsehwhere in GCC).

	gcc/rust/
	* rust-lang.cc (grs_langhook_type_for_mode): Placate clang-format.
CohenArthur pushed a commit to CohenArthur/gccrs that referenced this issue Jan 12, 2024
…_for_mode'

Yesterday's commit 3757e8d
"Merge commit '6baa95c9c5b3fea96fd22d03d961db4e4cf48d88' into HEAD [Rust-GCC#2112]"
brought in the Rust-GCC#1713 change from GCC upstream master branch that broke the
rules (via code copied verbatim from elsehwhere in GCC).

	gcc/rust/
	* rust-lang.cc (grs_langhook_type_for_mode): Placate clang-format.
CohenArthur pushed a commit to CohenArthur/gccrs that referenced this issue Jan 16, 2024
…_for_mode'

Yesterday's commit 3757e8d
"Merge commit '6baa95c9c5b3fea96fd22d03d961db4e4cf48d88' into HEAD [Rust-GCC#2112]"
brought in the Rust-GCC#1713 change from GCC upstream master branch that broke the
rules (via code copied verbatim from elsehwhere in GCC).

	gcc/rust/
	* rust-lang.cc (grs_langhook_type_for_mode): Placate clang-format.
CohenArthur pushed a commit to CohenArthur/gccrs that referenced this issue Jan 16, 2024
…_for_mode'

Yesterday's commit 3757e8d
"Merge commit '6baa95c9c5b3fea96fd22d03d961db4e4cf48d88' into HEAD [Rust-GCC#2112]"
brought in the Rust-GCC#1713 change from GCC upstream master branch that broke the
rules (via code copied verbatim from elsehwhere in GCC).

	gcc/rust/
	* rust-lang.cc (grs_langhook_type_for_mode): Placate clang-format.
CohenArthur pushed a commit to CohenArthur/gccrs that referenced this issue Jan 17, 2024
…_for_mode'

Yesterday's commit 3757e8d
"Merge commit '6baa95c9c5b3fea96fd22d03d961db4e4cf48d88' into HEAD [Rust-GCC#2112]"
brought in the Rust-GCC#1713 change from GCC upstream master branch that broke the
rules (via code copied verbatim from elsehwhere in GCC).

	gcc/rust/
	* rust-lang.cc (grs_langhook_type_for_mode): Placate clang-format.
XYenChi pushed a commit to XYenChi/gcc that referenced this issue Mar 6, 2024
As reported in "Rust related issues" at the end of
<https://inbox.sourceware.org/gcc-patches/20221219212344.ewtuzd2jtztm77x2@lug-owl.de>
"Modula-2 / Rust: Many targets failing",
<Rust-GCC/gccrs#1713>
"Test failure on msp430-elfbare target", for '--target=msp430-elfbar' we ICE:

    <built-in>: internal compiler error: Segmentation fault
    0xf2efbf crash_signal
          ../../gcc/gcc/toplev.cc:314
    0x120c8c7 build_function_type(tree_node*, tree_node*, bool)
          ../../gcc/gcc/tree.cc:7360
    0x120cc20 build_function_type_list(tree_node*, ...)
          ../../gcc/gcc/tree.cc:7442
    0x120d16b build_common_builtin_nodes()
          ../../gcc/gcc/tree.cc:9883
    0x8449b4 grs_langhook_init
          ../../gcc/gcc/rust/rust-lang.cc:132
    [...]

This is due to 'tmp == NULL' for 'gcc/tree.cc:build_common_builtin_nodes':

    tmp = lang_hooks.types.type_for_mode (targetm.eh_return_filter_mode (), 0);

..., were 'targetm.eh_return_filter_mode' (that is,
'gcc/targhooks.cc:default_eh_return_filter_mode',
via calling 'targetm.unwind_word_mode':
'gcc/config/msp430/msp430.cc:msp430_unwind_word_mode') returns 'PSImode',
and 'lang_hooks.types.type_for_mode' (that is,
'gcc/rust/rust-lang.cc:grs_langhook_type_for_mode') returns 'NULL' for
'PSImode'.

As, for example, discussed in <https://gcc.gnu.org/PR46805>
"ICE: SIGSEGV in optab_for_tree_code (optabs.c:407) with -O -fno-tree-scev-cprop -ftree-vectorize",
we have to support "random" modes/types in the 'type_for_mode' langhook.

	gcc/rust/
	* rust-lang.cc (grs_langhook_type_for_mode): Also consider all
	'int_n' modes/types.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Projects
Status: Done
Development

No branches or pull requests

3 participants