On 1/5/21 12:46 AM, Alexandre Oliva wrote:
> In g++.dg/opt/store-merging-2.C, the natural alignment of types T and
> S is a single byte, so we shouldn't expect store merging on
> strict-alignment platforms.  Indeed, without something like the
> adjust-alignment pass to bump up the alignment of the automatic
> variable, as in GCC 10, the optimization does not occur.
>
> This patch adjusts the test so that the required alignment is
> expressly stated, and so we don't rely on its accidentally being there
> to get the desired optimization.
>
> Regstrapped on x86_64-linux-gnu, also tested on x-arm-wrs-vxworks7r2.
> Ok to install?
>
>
> for  gcc/testsuite/ChangeLog
>
>       * g++.dg/opt/store-merging-2.C: Add the required alignment.
OK
jeff

Reply via email to