This is the mail archive of the
gcc-bugs@gcc.gnu.org
mailing list for the GCC project.
[Bug tree-optimization/65215] [5 Regression] Bswap load miscompilation
- From: "jakub at gcc dot gnu.org" <gcc-bugzilla at gcc dot gnu dot org>
- To: gcc-bugs at gcc dot gnu dot org
- Date: Thu, 26 Feb 2015 11:36:41 +0000
- Subject: [Bug tree-optimization/65215] [5 Regression] Bswap load miscompilation
- Auto-submitted: auto-generated
- References: <bug-65215-4 at http dot gcc dot gnu dot org/bugzilla/>
https://gcc.gnu.org/bugzilla/show_bug.cgi?id=65215
--- Comment #8 from Jakub Jelinek <jakub at gcc dot gnu.org> ---
(In reply to Thomas Preud'homme from comment #7)
> (In reply to Jakub Jelinek from comment #6)
> > I can certainly remove that hunk from the patch, if the expander and other
> > passes handle it well. The test can stay I guess.
>
> Things are at least working on x86 (obviously), ARM and SPARC (after
> PR61320's fix). Also this code is there since a long time without any bug
> report problems due to unalignment.
Ok.
> One question about the patch: is there a reason not to use n->range instead
> of GET_MODE_BITSIZE (TYPE_MODE (load_type))?
Supposedly that can be used too, that should probably always be the case.