On Wed, Sep 19, 2012 at 09:04:30PM +0200, Dag-Erling Sm??rgrav wrote:
> Konstantin Belousov writes:
> > "Dag-Erling Sm??rgrav" writes:
> > > I assume you mean assignments, not calculations. I trust the compiler
> > > to move them to where they are needed - a trivial optimization with SSA.
> > It
Konstantin Belousov writes:
> "Dag-Erling Smørgrav" writes:
> > I assume you mean assignments, not calculations. I trust the compiler
> > to move them to where they are needed - a trivial optimization with SSA.
> It is a dereference before the assignment, so I perceive it as the
> calculation. N
On Wed, Sep 19, 2012 at 02:58:02PM +0200, Dag-Erling Sm??rgrav wrote:
> Konstantin Belousov writes:
> > "Dag-Erling Sm??rgrav" writes:
> > > + otable = fdp->fd_ofiles;
> > > + ofileflags = fdp->fd_ofileflags;
> > These two new calculations could be unused if the function return early.
>
> I assu
Konstantin Belousov writes:
> "Dag-Erling Smørgrav" writes:
> > + otable = fdp->fd_ofiles;
> > + ofileflags = fdp->fd_ofileflags;
> These two new calculations could be unused if the function return early.
I assume you mean assignments, not calculations. I trust the compiler
to move them to
On Tue, Sep 18, 2012 at 05:46:23PM +0200, Dag-Erling Sm??rgrav wrote:
> The patch below my signature improves the legibility of fdgrowtable(),
> and adds comments explaining the hairier bits. The only functional
> change is that the code no longer overwrites the old fileflags array
> when the old
The patch below my signature improves the legibility of fdgrowtable(),
and adds comments explaining the hairier bits. The only functional
change is that the code no longer overwrites the old fileflags array
when the old table is placed on the freelist; instead, it uses the space
actually set aside
6 matches
Mail list logo