https://gcc.gnu.org/bugzilla/show_bug.cgi?id=89497

--- Comment #24 from Richard Biener <rguenth at gcc dot gnu.org> ---
(In reply to Jakub Jelinek from comment #23)
> Created attachment 45926 [details]
> rh1686696.c
> 
> r269302 also fixed the following testcase that has been miscompiled on
> s390x-linux with -march=zEC12 -O2 -fno-strict-aliasing at least (or
> -march=zEC12 -O2, -fno-strict-aliasing just because the testcase isn't
> completely kosher, as it initializes malloced memory through long long array
> and then accesses it as a struct containing long long and long long array
> fields).
> Is it worth adding this into the testsuite?

Hmm, the patch doesn't contain a wrong-code fix - at least not that I know of.
As we've seen it may change generated code so are you sure the reason the
testcase was miscompiled didn't go latent?

Reply via email to