[LLVMdev] Complex load patterns and token factors

Hal Finkel hfinkel at anl.gov
Sun Jun 24 10:28:43 PDT 2012


On Sun, 24 Jun 2012 18:09:48 +0100
Tim Northover <t.p.northover at gmail.com> wrote:

> > If it is not being used, maybe we should change it to be
> > doesNotNeedChain?
> 
> That sounded like a really good idea until I started to actually go
> through removing all traces of hasCtrlDep from other Targets. I felt
> really bad about wiping those flags people had carefully put on their
> instructions, even if useless.
> 
> > I am not sure what kind of load does not need to be chained
> > (prefetch?), but I'd prefer to keep the system friendly if at all
> > possible. They'll be fewer bugs that way.
> 
> That is the main worry. Could we justify a second "hasNoCtrlDep" in
> addition to "hasCtrlDep", with it being a TableGen error to set both?
> Or just forget about the override until someone actually comes up with
> an instruction that's legitimately "mayLoad" or whatever yet doesn't
> want a chain

This is probably the best approach. There is no need to invent a
feature with no use cases.

 -Hal

> (I don't think even prefetch counts -- if other
> instructions are modifying that location it needs to know).
> 
> > Thanks again,
> 
> No worries.
> 
> Tim.



-- 
Hal Finkel
Postdoctoral Appointee
Leadership Computing Facility
Argonne National Laboratory



More information about the llvm-dev mailing list