Hi Grischka,

Your last commit reverted alignment code introduced to support OpenBSD on
arm.

Even it looks strange for you, as confirmed by an OpenBSD developer, arm by
design requires alignment.

Your commit breaks the test suite:

Test: 96_nodata_wanted...

Bus error (core dumped)

--- 96_nodata_wanted.expect     Mon Jan 25 00:56:49 2021

+++ 96_nodata_wanted.output     Mon Jan 25 01:34:37 2021

@@ -12,15 +12,3 @@

96_nodata_wanted.c:25: warning: cast between pointer and integer of
different size

 

[test_data_suppression_off]

-data:

-  8 - 8.0 - 8.0 - main - static string

-  333 44 555555 6 7

-size of data/text:

-  non-zero/non-zero

-

-[test_data_suppression_on]

-size of data/text:

-  zero/zero

-

-[test_static_data]

-a = 1

 

 

FYI, with Herman (esp. Herman) we added OpenBSD support for
i386/x86_64/arm/arm64 which passed all the test. The good news is that tcc
is now an official OpenBSD package which is great for the TinyCC popularity.

If, for probably good reasons, you don't want alignment code for all
platforms, can we add a new configuration option: --align or alike which
will force alignment?

 

Christian

_______________________________________________
Tinycc-devel mailing list
Tinycc-devel@nongnu.org
https://lists.nongnu.org/mailman/listinfo/tinycc-devel

Reply via email to