On Sun, Nov 11, 2018 at 7:55 AM Akira Yokosawa <akiyks@xxxxxxxxx> wrote:
>
> On 2018/11/10 20:49:58 +0800, Junchang Wang wrote:
> > Signed-off-by: Junchang Wang <junchangwang@xxxxxxxxx>
> > ---
> > Hi Paul,
> >
> > This is the only patch for Chapter Tools of Trade. Please take a look.
> >
> >
> > Thanks,
> > --Junchang
> >
> > --
> > toolsoftrade/toolsoftrade.tex | 4 ++--
> > 1 file changed, 2 insertions(+), 2 deletions(-)
> >
> > diff --git a/toolsoftrade/toolsoftrade.tex b/toolsoftrade/toolsoftrade.tex
> > index 0844b4c..b4bc46a 100644
> > --- a/toolsoftrade/toolsoftrade.tex
> > +++ b/toolsoftrade/toolsoftrade.tex
> > @@ -2043,8 +2043,8 @@ machine-sized and properly aligned.
> > It can also prevent load fusing, store fusing, invented loads, and
> > invented stores.
> > However, although it does prevent the compiler from reordering \co{volatile}
> > -accesses with each other, does nothing to prevent the CPU from reordering
> > -these accesses.
> > +accesses with each other, \co{volatile} keyword does nothing to prevent the
> > +CPU from reordering these accesses.
>
> In this context, I think
>
> +accesses with each other, it does nothing to prevent the CPU from reordering
> these accesses.
>
> should be good enough.
Hi Akira,
After reading the whole paragraph a few times, I realized you are
right :-). Thanks a lot!
--Junchang
>
> Thanks, Akira
>
> > Furthermore, it does nothing to prevent either compiler or CPU from
> > reordering non-\co{volatile} accesses with each other or with
> > \co{volatile} accesses.
> >
>
