Change to readonly conflict handling
Richard Kenner
kenner@vlsi1.ultra.nyu.edu
Sun Sep 17 17:16:00 GMT 2000
This code is actually not valid C; neither of the two assignments are
valid, because they change p->a and q->a which are 'const'.
Yes, I know, but I don't think there's any way to get it from valid C. This
was meant to be as close I could get the actual case, which was from Ada.
Actually, what I get is _both_ words of *p and *q being stored with
/u. On powerpc (so the fields are SImode):
Right, I get that too. I even think I see why, though it is a bug.
(insn 16 15 19 (set (mem/s/u:DI (reg/v:SI 82) 0)
(reg:DI 86)) -1 (nil)
(nil))
(insn 19 16 20 (set (reg:DI 87)
(mem/s:DI (reg/v:SI 82) 0)) -1 (nil)
(nil))
But *this* is key: it's being stored *with* the "/u", but loaded without.
That will cause these two loads to be marked non-conflicting, which may
have them scheduled in the wrong order (how I ran into the bug).
More information about the Gcc
mailing list