public inbox for gcc-bugs@sourceware.org
help / color / mirror / Atom feed
* [Bug c/110234] New: [RISCV] Improper endian encoding when using floats and doubles with -mbig-endian
@ 2023-06-13 7:44 branislav.brzak at syrmia dot com
2023-06-13 7:45 ` [Bug c/110234] " branislav.brzak at syrmia dot com
` (2 more replies)
0 siblings, 3 replies; 4+ messages in thread
From: branislav.brzak at syrmia dot com @ 2023-06-13 7:44 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110234
Bug ID: 110234
Summary: [RISCV] Improper endian encoding when using floats and
doubles with -mbig-endian
Product: gcc
Version: 12.2.0
Status: UNCONFIRMED
Severity: normal
Priority: P3
Component: c
Assignee: unassigned at gcc dot gnu.org
Reporter: branislav.brzak at syrmia dot com
Target Milestone: ---
Problem description
RISCV Gcc has a -mbig-endian flag that is supposed to flip endian on every
available data type, except instructions stored in `.text`. However, there is
an issue with how floats and doubles are stored. With the following min repro:
.section .text
.global _start
_start:
j _start
.section .data
float1: .float 1.1
That is compiled with the following makefile:
all:
riscv64-unknown-linux-gnu-gcc -ffreestanding -nostdlib -march=rv64imafd
-mno-riscv-attribute -mabi=lp64d -Tlink.ld -nostartfiles -mcmodel=medany
-Wl,--no-warn-rwx-segments -g -o le_test.elf test.s
riscv64-unknown-linux-gnu-gcc -ffreestanding -nostdlib -march=rv64imafd
-mno-riscv-attribute -mabi=lp64d -Tlink.ld -nostartfiles -mcmodel=medany
-Wl,--no-warn-rwx-segments -mbig-endian -g -o be_test.elf test.s
riscv64-unknown-elf-objdump -D le_test.elf > le_test.dump
riscv64-unknown-elf-objdump -D be_test.elf > be_test.dump
riscv64-unknown-linux-gnu-objcopy -O binary le_test.elf le_test.bin
riscv64-unknown-linux-gnu-objcopy -O binary be_test.elf be_test.bin
xxd le_test.bin > le_test.hex
xxd be_test.bin > be_test.hex
diff:
@diff le_test.hex be_test.hex
Doing `make diff` on the resulting hex dumps gives:
1c1
< 00000000: 6f00 0000 0000 0000 cdcc 8c3f 0000 0000 o..........?....
---
> 00000000: 6f00 0000 0000 0000 cccd 3f8c 0000 0000 o.........?.....
Upper one is the LE binary while the lower one is BE. Here we can see that
compared to LE (that is properly encoded), BE's endian is flipped only on 2
byte boundaries. The correct BE sequence in this case would be
3f 8c cc cd
2 byte stores are also reflected in the elf file as well:
GCC version info
riscv64-unknown-linux-gnu-gcc -v
Using built-in specs.
COLLECT_GCC=riscv64-unknown-linux-gnu-gcc
COLLECT_LTO_WRAPPER=/opt/riscv/libexec/gcc/riscv64-unknown-linux-gnu/12.2.0/lto-wrapper
Target: riscv64-unknown-linux-gnu
Configured with: /home/fix/tasks/riscvbe/new/riscv-gnu-toolchain/gcc/configure
--target=riscv64-unknown-linux-gnu --prefix=/opt/riscv
--with-sysroot=/opt/riscv/sysroot --with-pkgversion=g2ee5e430018
--with-system-zlib --enable-shared --enable-tls
--enable-languages=c,c++,fortran --disable-libmudflap --disable-libssp
--disable-libquadmath --disable-libsanitizer --disable-nls --disable-bootstrap
--src=.././gcc --disable-multilib --with-abi=lp64d --with-arch=rv64imafdc
--with-tune=rocket --with-isa-spec=20191213 'CFLAGS_FOR_TARGET=-O2
-mcmodel=medlow' 'CXXFLAGS_FOR_TARGET=-O2 -mcmodel=medlow'
Thread model: posix
Supported LTO compression algorithms: zlib
gcc version 12.2.0 (g2ee5e430018)
^ permalink raw reply [flat|nested] 4+ messages in thread
* [Bug c/110234] [RISCV] Improper endian encoding when using floats and doubles with -mbig-endian
2023-06-13 7:44 [Bug c/110234] New: [RISCV] Improper endian encoding when using floats and doubles with -mbig-endian branislav.brzak at syrmia dot com
@ 2023-06-13 7:45 ` branislav.brzak at syrmia dot com
2023-06-13 12:32 ` [Bug target/110234] " pinskia at gcc dot gnu.org
2023-06-13 13:34 ` xry111 at gcc dot gnu.org
2 siblings, 0 replies; 4+ messages in thread
From: branislav.brzak at syrmia dot com @ 2023-06-13 7:45 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110234
--- Comment #1 from Branislav Brzak <branislav.brzak at syrmia dot com> ---
For "2 byte stores are also reflected in the elf file as well:" I wanted to
include this:
0000000080000008 <_sdata>:
80000008: cccd .2byte 0xcccd
8000000a: 3f8c .2byte 0x3f8c
^ permalink raw reply [flat|nested] 4+ messages in thread
* [Bug target/110234] [RISCV] Improper endian encoding when using floats and doubles with -mbig-endian
2023-06-13 7:44 [Bug c/110234] New: [RISCV] Improper endian encoding when using floats and doubles with -mbig-endian branislav.brzak at syrmia dot com
2023-06-13 7:45 ` [Bug c/110234] " branislav.brzak at syrmia dot com
@ 2023-06-13 12:32 ` pinskia at gcc dot gnu.org
2023-06-13 13:34 ` xry111 at gcc dot gnu.org
2 siblings, 0 replies; 4+ messages in thread
From: pinskia at gcc dot gnu.org @ 2023-06-13 12:32 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110234
--- Comment #2 from Andrew Pinski <pinskia at gcc dot gnu.org> ---
Note this looks like an assembler bug rather than a gcc bug.
Assembler in your case comes from the binutils project who's bugzilla is
located at https://sourceware.org/bugzilla.
^ permalink raw reply [flat|nested] 4+ messages in thread
* [Bug target/110234] [RISCV] Improper endian encoding when using floats and doubles with -mbig-endian
2023-06-13 7:44 [Bug c/110234] New: [RISCV] Improper endian encoding when using floats and doubles with -mbig-endian branislav.brzak at syrmia dot com
2023-06-13 7:45 ` [Bug c/110234] " branislav.brzak at syrmia dot com
2023-06-13 12:32 ` [Bug target/110234] " pinskia at gcc dot gnu.org
@ 2023-06-13 13:34 ` xry111 at gcc dot gnu.org
2 siblings, 0 replies; 4+ messages in thread
From: xry111 at gcc dot gnu.org @ 2023-06-13 13:34 UTC (permalink / raw)
To: gcc-bugs
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=110234
Xi Ruoyao <xry111 at gcc dot gnu.org> changed:
What |Removed |Added
----------------------------------------------------------------------------
Resolution|--- |MOVED
Status|UNCONFIRMED |RESOLVED
CC| |xry111 at gcc dot gnu.org
--- Comment #3 from Xi Ruoyao <xry111 at gcc dot gnu.org> ---
Not a GCC bug. In this case GCC is only a wrapper of the assembler. GCC even
does not know how to parse the assembly.
^ permalink raw reply [flat|nested] 4+ messages in thread
end of thread, other threads:[~2023-06-13 13:34 UTC | newest]
Thread overview: 4+ messages (download: mbox.gz / follow: Atom feed)
-- links below jump to the message on this page --
2023-06-13 7:44 [Bug c/110234] New: [RISCV] Improper endian encoding when using floats and doubles with -mbig-endian branislav.brzak at syrmia dot com
2023-06-13 7:45 ` [Bug c/110234] " branislav.brzak at syrmia dot com
2023-06-13 12:32 ` [Bug target/110234] " pinskia at gcc dot gnu.org
2023-06-13 13:34 ` xry111 at gcc dot gnu.org
This is a public inbox, see mirroring instructions
for how to clone and mirror all data and code used for this inbox;
as well as URLs for read-only IMAP folder(s) and NNTP newsgroup(s).