diff options
author | Tom de Vries <tdevries@suse.de> | 2020-09-30 11:51:41 +0200 |
---|---|---|
committer | Tom de Vries <tdevries@suse.de> | 2020-09-30 14:19:08 +0200 |
commit | 4c0eb14bc8553d6e97f1cf70276c6a4fa67b7c38 (patch) | |
tree | f139f72cb9526b892e6d7d5a35d1abf4f287a02c /gcc/fortran/trans-expr.c | |
parent | 92e652d8c21bd7e66cbb0f9001542a2f55345af0 (diff) | |
download | gcc-4c0eb14bc8553d6e97f1cf70276c6a4fa67b7c38.zip gcc-4c0eb14bc8553d6e97f1cf70276c6a4fa67b7c38.tar.gz gcc-4c0eb14bc8553d6e97f1cf70276c6a4fa67b7c38.tar.bz2 |
[testsuite] Re-enable pr94600-{1,3}.c tests for arm
Before commit 7e437162001 "[testsuite] Require non_strict_align in
pr94600-{1,3}.c", some tests were failing for nvptx, because volatile stores
were expected, but memcpy's were found instead.
This was traced back to this bit in compute_record_mode:
...
/* If structure's known alignment is less than what the scalar
mode would need, and it matters, then stick with BLKmode. */
if (mode != BLKmode
&& STRICT_ALIGNMENT
&& ! (TYPE_ALIGN (type) >= BIGGEST_ALIGNMENT
|| TYPE_ALIGN (type) >= GET_MODE_ALIGNMENT (mode)))
{
/* If this is the only reason this type is BLKmode, then
don't force containing types to be BLKmode. */
TYPE_NO_FORCE_BLK (type) = 1;
mode = BLKmode;
}
...
which got triggered for nvptx, but not for x86_64.
The commit disabled the tests for non_strict_align effective target, but
that had the effect for the arm target that those tests were disabled, even
though they were passing before.
Further investigation in compute_record_mode shows that the if-condition
evaluates to false for arm because, because TYPE_ALIGN (type) == 32, while
it's 8 for nvptx. This again can be explained by the
PCC_BITFIELD_TYPE_MATTERS setting, which is 1 for arm, but 0 for nvptx.
Re-enable the test for arm by using effective target
(non_strict_align || pcc_bitfield_type_matters).
Tested on arm-eabi and nvptx.
gcc/testsuite/ChangeLog:
2020-09-30 Tom de Vries <tdevries@suse.de>
* gcc.dg/pr94600-1.c: Use effective target
(non_strict_align || pcc_bitfield_type_matters).
* gcc.dg/pr94600-3.c: Same.
Diffstat (limited to 'gcc/fortran/trans-expr.c')
0 files changed, 0 insertions, 0 deletions