On Mon, 15 Aug 2016, ron minnich wrote:
> Just one minor change, I had to addidentifier x;
Yeah, sure. I wasn't paying enough attenton to realize that x was
probably not the field name you had in mind :)
julia
>
> and then it works. Many thanks, as always, for spatch.
> ron
>
> On Mon, Aug 15, 2016 at 10:29 AM Julia Lawall <[email protected]> wrote:
>
>
> On Mon, 15 Aug 2016, ron minnich wrote:
>
> > I have this struct:
> > struct ahba {
> > uint32_t x;
> > }
> >
> > given this:
> > struct ahba *h;
> >
> > I want to convert derefs of h->x to turn into
> mmio_read32(&h->x);
> > h->x = y;
> > to
> > mmio_write32(&h->x, y);
>
> What semantic patch did you write? It looks like it should be:
>
> @@
> struct ahba *h;
> expression e;
> @@
>
> - h->x = e
> + mmio_write32(&h->x,e)
>
> @@
> struct ahba *h;
> @@
>
> - h->x
> + mmio_read32(&h->x)
>
> The order is important.
>
> julia
>
>
>_______________________________________________
Cocci mailing list
[email protected]
https://systeme.lip6.fr/mailman/listinfo/cocci
