This is the mail archive of the gcc-patches@gcc.gnu.org mailing list for the GCC project.

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]
Other format:	[Raw text]

RE: [PATCH][2/3] Fix PR54733 Optimize endian independent load/store

From: "Thomas Preud'homme" <thomas dot preudhomme at arm dot com>
To: "GCC Patches" <gcc-patches at gcc dot gnu dot org>
Date: Sun, 4 May 2014 17:51:22 +0800
Subject: RE: [PATCH][2/3] Fix PR54733 Optimize endian independent load/store
Authentication-results: sourceware.org; auth=none
References: <000b01cf4e0e$54044b70$fc0ce250$ at arm dot com> <alpine dot DEB dot 2 dot 10 dot 1404020800200 dot 3584 at laptop-mg dot saclay dot inria dot fr> <001301cf4e41$d2a4f320$77eed960$ at arm dot com> <CAFiYyc2WTsb8CPzQK7m_aqcsy5Rjbw_2HWVJO8CVxFha_+Mkfg at mail dot gmail dot com> <003001cf4eff$74b0e210$5e12a630$ at arm dot com> <003c01cf4fc9$a0b37510$e21a5f30$ at arm dot com> <CAFiYyc2624v+tN6JskOvgjWf7B8vtKZTyvB6s3VHc_-vo5=t7w at mail dot gmail dot com> <000001cf59fc$a512b110$ef381330$ at arm dot com> <CAFiYyc0M_+5uY9O-JAof7+Xt6FtGS-XkUb47R1wSmR=d9_hJ9Q at mail dot gmail dot com> <002801cf5f5d$58cbc420$0a634c60$ at arm dot com> <CAFiYyc1AuBvG3rGtmVE32dS-DRawQyhEg9GcqKcUB9C-2VDVWg at mail dot gmail dot com>

Please find attached the new version of this patch addressing all your comments.

ChangeLog are now as follows:

*** gcc/ChangeLog ***

2014-05-04  Thomas Preud'homme  <thomas.preudhomme@arm.com>

	PR tree-optimization/54733
	* expr.c (get_inner_reference): Add a parameter to control whether a
	MEM_REF should be split into base + offset.
	* tree.h (get_inner_reference): Default new parameter to false.
	* tree-ssa-math-opts.c (find_bswap_load): New.
	(find_bswap_1): Add support for memory source.
	(find_bswap): Likewise.
	(execute_optimize_bswap): Likewise.

*** gcc/testsuite/ChangeLog ***

2014-05-04  Thomas Preud'homme  <thomas.preudhomme@arm.com>

	PR tree-optimization/54733
	* gcc.dg/optimize-bswapdi-3.c: New test to check extension of bswap
	optimization to support memory sources.
	* gcc.dg/optimize-bswaphi-1.c: Likewise.
	* gcc.dg/optimize-bswapsi-2.c: Likewise.
	* gcc.c-torture/execute/bswap-2.c: Likewise.

Ok for stage1? Note that I haven't commited part 1 despite your approval as I'm waiting for the other parts to be approved. I don't know if it's the usual procedure for patchset.

Best regards,

Thomas Preud'homme

Attachment: gcc32rm-84.5.1.part2.diff
Description: Binary data

Follow-Ups:
- RE: [PATCH][2/3] Fix PR54733 Optimize endian independent load/store
  - From: Thomas Preud'homme

Index Nav:	[Date Index] [Subject Index] [Author Index] [Thread Index]
Message Nav:	[Date Prev] [Date Next]	[Thread Prev] [Thread Next]