[PATCH] D23876: [CFLAA] On-Demand variant of Context Free Language Alias Analysis

[Daniel Berlin via llvm-commits]

On Thu, Aug 25, 2016 at 10:00 AM, Hal Finkel <hfinkel at anl.gov> wrote:

>
> ------------------------------
>
> *From: *"Daniel Berlin via llvm-commits" <llvm-commits at lists.llvm.org>
> *To: *reviews+D23876+public+4a33dfaeb73cf394 at reviews.llvm.org, "David
> Callahan" <dcallahan at fb.com>
> *Cc: *"llvm-commits" <llvm-commits at lists.llvm.org>
> *Sent: *Thursday, August 25, 2016 11:33:32 AM
> *Subject: *Re: [PATCH] D23876: [CFLAA] On-Demand variant of Context Free
> Language Alias Analysis
>
> I'll take a glance at this, however, to bikeshed something, it probably
> shouldn't be called "on-demand".
>
> In an ideal world, we should name these things by their precision vs speed
> tradeoff or something, so someone has some hope of knowing which to choose.
>
> Especially if we have *3* of them :)
>
> Obviously we need to think about how we want this to work :)
>

Yeah.
In an ideal world, we should expose one thing, and internally,  it uses
some combination of these passes to provide good results for small and
large problems, ....


> -- It is not clear to me that we want 3 of them. I imagine that we want an
> analysis that is precise for small problems and then degrades gracefully as
> the problems get larger (some some appropriate definitions of smaller and
> larger). On-demand updating is often preferable regardless (whenever doing
> so is cheaper than redoing everything).
>
>
I think we are both on the same page here :)


>  -Hal
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>
>
>
>
> --
> Hal Finkel
> Assistant Computational Scientist
> Leadership Computing Facility
> Argonne National Laboratory
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160825/de8f5520/attachment.html>