This is the mail archive of the
mailing list for the GCC project.
Re: [SH] PR 52483 - Fix volatile mem stores
- From: Kaz Kojima <kkojima at rr dot iij4u dot or dot jp>
- To: oleg dot endo at t-online dot de
- Cc: gcc-patches at gcc dot gnu dot org, kkojima at gcc dot gnu dot org
- Date: Fri, 25 Oct 2013 12:31:02 +0900 (JST)
- Subject: Re: [SH] PR 52483 - Fix volatile mem stores
- Authentication-results: sourceware.org; auth=none
- References: <1371897400 dot 3742 dot 30 dot camel at yam-132-YW-E178-FTW> <1382472552 dot 2445 dot 137 dot camel at yam-132-YW-E178-FTW>
Oleg Endo <firstname.lastname@example.org> wrote:
> The attached patch fixes volatile mem stores on SH so that they won't
> result in redundant sign/zero extensions and will utilize available
> addressing modes. This is similar to what has been done to fix memory
> loads in http://gcc.gnu.org/ml/gcc-patches/2013-06/msg01315.html
> Tested on rev 203909 with
> make -k check RUNTESTFLAGS="--target_board=sh-sim
> and no new failures.
> However, the test result summary showed a bunch of "WARNING: program
> timed out." Kaz, could you please add it to your test queue and let me
> know if it's OK for trunk?
There are no new failures on my sh4-unknown-linux-gnu tests and
I can't see extra "program timed out" cases for my sh-elf build
on unified tree.
The patch itself looks fine and Ok for trunk.