On Thu, Oct 22, 2015 at 09:49:29AM +0200, Richard Biener wrote:
> >> Jakub, IYR I originally had IFN_FORK and IFN_JOIN as such distinct internal
> >> fns. This replaces that scheme.
> >>
> >> ok?
> >
> > Hmm, I'd just have used gimple_has_volatile_ops on the call? That
> > should have the
> > desired effects.
>
> That is, whatever new IFNs you need are ok, but special-casing them is not
> necessary if you properly mark the calls as volatile.
I don't see gimple_has_volatile_ops used in tracer.c or
tree-ssa-threadedge.c. Setting gimple_has_volatile_ops on those IFNs is
fine, but I think they are even stronger than that.
Jakub
