[PATCH] expose ILP for associative operations in the DAG

James Molloy james at jamesmolloy.co.uk
Thu May 14 13:24:39 PDT 2015


Hi,

It sounds like the best time to do this if at all possible is in
MachineInst form, where we have MachineTraceMetrics and accurate register
pressure information.

Is there a reason why you're not doing it there? I know it's more awkward,
but it really seems we need accurate register pressure if we're not going
to make it cripplingly conservative.

Cheers,

James

On Thu, 14 May 2015 at 21:17 Fiona Glaser <escha at apple.com> wrote:

> Could you help me try to understand what purpose this serves in terms of
> complementing the already-existing reassociate pass? That is, what benefit
> comes from doing it in the DAG in addition to the existing reassociate IR
> pass?
>
>
> http://reviews.llvm.org/D9780
>
> EMAIL PREFERENCES
>   http://reviews.llvm.org/settings/panel/emailpreferences/
>
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20150514/3f77260a/attachment.html>


More information about the llvm-commits mailing list