This is the mail archive of the
gcc-patches@gcc.gnu.org
mailing list for the GCC project.
Re: Fix PR53908
- From: Hans-Peter Nilsson <hans-peter dot nilsson at axis dot com>
- To: rsandifo at nildram dot co dot uk
- Cc: stevenb dot gcc at gmail dot com, bernds at codesourcery dot com, gcc-patches at gcc dot gnu dot org, hp at bitrange dot com
- Date: Fri, 13 Jul 2012 08:47:42 +0200
- Subject: Re: Fix PR53908
> From: Richard Sandiford <rsandifo@nildram.co.uk>
> Date: Thu, 12 Jul 2012 21:18:54 +0200
> if (CALL_P (insn))
> {
> if (RTL_CONST_OR_PURE_CALL_P (insn))
> /* Pure functions can read from memory. Const functions can
> read from arguments that the ABI has forced onto the stack.
> Neither sort of read can be volatile. */
> memrefs_in_across |= MEMREF_NORMAL;
> else
> {
> memrefs_in_across |= MEMREF_VOLATILE;
> mem_sets_in_across |= MEMREF_VOLATILE;
> }
> }
>
> OK with that change if you agree.
(Steven or Bernd, please ACK/NAK, for quick resolution.)
I've tested this version for trunk; no regressions for revision
189439 on x86_64-unknown-linux-gnu (N.B. the test-case doesn't
trigger on trunk). I've also tested Steven's earlier version,
same results, if you change your mind. :)
It'll be another 6:21 hours(!) before results from the 4.7
branch.
brgds, H-P