This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: PATCH: PR target/64905: unsigned short is loaded with 4-byte load (movl)
- From: Uros Bizjak <ubizjak at gmail dot com>
- To: "H.J. Lu" <hjl dot tools at gmail dot com>
- Cc: "gcc-patches at gcc dot gnu dot org" <gcc-patches at gcc dot gnu dot org>, Jan Hubicka <hubicka at ucw dot cz>
- Date: Wed, 4 Feb 2015 14:21:21 +0100
- Subject: Re: PATCH: PR target/64905: unsigned short is loaded with 4-byte load (movl)
- Authentication-results: sourceware.org; auth=none
- References: <20150202213947 dot GA27865 at intel dot com>
On Mon, Feb 2, 2015 at 10:39 PM, H.J. Lu <hongjiu.lu@intel.com> wrote:
> This patch fixes a long standing bug where aligned_operand ignores
> alignment of memory operand less than 32 bits. It drops address
> decomposition and returns false if alignment of memory operand less
> is than 32 bits. Tested on Linux/x86-64. OK for trunk, 4.9 and 4.8
> branches?
Can you please find some references in gcc mainlig lists why and for
what reason is the predicate written in the current way? Are there
some (older?) processors that require this approach, so a tuning flag
should be used here?
OTOH, this is not a regression, so not a stage-4 material.
Uros.