llvm dev - Mar 2019 - Interpreter improvement

Hi,
I going through the Interpreter class source code and I think making almost
all the visit methods (maybe not visitInstruction yet) would really help
anybody that wants to create any sort of tracer or similar. Would be a
patch in this case worthy?

Thanks
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<http://lists.llvm.org/pipermail/llvm-dev/attachments/20190304/01428165/attachment.html>

I dont quite understand the meaning of making almost all the visit methods, care
to explain?

On a side note, last time I checked that part I think it probably needs a full
rewrite from ground up IMHO

Zhang
> 在 2019年3月4日，08:06，Alberto Barbaro via llvm-dev <llvm-dev at
lists.llvm.org> 写道：
> 
> Hi,
> I going through the Interpreter class source code and I think making almost
all the visit methods (maybe not visitInstruction yet) would really help anybody
that wants to create any sort of tracer or similar. Would be a patch in this
case worthy?
> 
> Thanks
> _______________________________________________
> LLVM Developers mailing list
> llvm-dev at lists.llvm.org
> https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev

Hi,
I my case I needed to trace the execution of a specific .ll file. I wanted
to know when a store and load instruction were executed. Maybe extending
the Interpreter class was not the best option so I'm happy to hear
suggestions.

As a curiosity, why do you think that class should be rewritten?

Thanks

On Mon, Mar 4, 2019, 08:15 mayuyu.io <admin at mayuyu.io> wrote:
> I dont quite understand the meaning of making almost all the visit
> methods, care to explain?
>
> On a side note, last time I checked that part I think it probably needs a
> full rewrite from ground up IMHO
>
> Zhang
>
> > 在 2019年3月4日，08:06，Alberto Barbaro via llvm-dev <llvm-dev at
lists.llvm.org>
> 写道：
> >
> > Hi,
> > I going through the Interpreter class source code and I think making
> almost all the visit methods (maybe not visitInstruction yet) would really
> help anybody that wants to create any sort of tracer or similar. Would be a
> patch in this case worthy?
> >
> > Thanks
> > _______________________________________________
> > LLVM Developers mailing list
> > llvm-dev at lists.llvm.org
> > https://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-dev
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL:
<http://lists.llvm.org/pipermail/llvm-dev/attachments/20190304/309ec43c/attachment.html>