
<html><head><style type='text/css'>p { margin: 0; }</style></head><body><div style='font-family: arial,helvetica,sans-serif; font-size: 10pt; color: #000000'><br><hr id="zwchr"><blockquote style="border-left: 2px solid rgb(16, 16, 255); margin-left: 5px; padding-left: 5px; color: rgb(0, 0, 0); font-weight: normal; font-style: normal; text-decoration: none; font-family: Helvetica,Arial,sans-serif; font-size: 12pt;"><b>From: </b>"Daniel Berlin" <dberlin@dberlin.org><br><b>To: </b>reviews+D12414+public+cf5673a649e54709@reviews.llvm.org<br><b>Cc: </b>"Jingyue Wu" <jingyue@google.com>, wengxt@gmail.com, "Chandler Carruth" <chandlerc@gmail.com>, "Justin Holewinski" <jholewinski@nvidia.com>, "Eli Bendersky" <eliben@google.com>, "Hal Finkel" <hfinkel@anl.gov>, "Bjarke Roune" <broune@google.com>, "Mark Heffernan" <meheff@google.com>, "llvm-commits" <llvm-commits@lists.llvm.org><br><b>Sent: </b>Tuesday, March 15, 2016 1:03:35 PM<br><b>Subject: </b>Re: [PATCH] D12414: [NVPTX] add an NVPTX-specific alias analysis<br><br><div dir="ltr">+1.<br><div>If we need this, we should make it work ;)<br><br></div><div>I'm not sure anyone sat down and said "it would be really bad to let targets have their own AA", and make such a decision</div><div>(if they did, they didn't discuss it on llvm-dev)</div><div id="DWT3767"><br></div></div></blockquote><br>I'm pretty sure I specifically requested that the infrastructure design not preclude this from working. We certainly don't want targets working around general AA deficiencies with target-specific passes, but for dealing with target-specific intrinsics and address spaces (etc.) I think it makes a lot of sense.<br><br> -Hal<br><br><blockquote style="border-left: 2px solid rgb(16, 16, 255); margin-left: 5px; padding-left: 5px; color: rgb(0, 0, 0); font-weight: normal; font-style: normal; text-decoration: none; font-family: Helvetica,Arial,sans-serif; font-size: 12pt;"><div dir="ltr"><div></div></div><div class="gmail_extra"><br><div class="gmail_quote">On Tue, Mar 15, 2016 at 10:57 AM, Hal Finkel <span dir="ltr"><<a href="mailto:hfinkel@anl.gov" target="_blank">hfinkel@anl.gov</a>></span> wrote:<br><blockquote class="gmail_quote" style="margin: 0pt 0pt 0pt 0.8ex; border-left: 1px solid rgb(204, 204, 204); padding-left: 1ex;">hfinkel added a comment.<br>

<span class=""><br>

In <a href="http://reviews.llvm.org/D12414#375657" rel="noreferrer" target="_blank">http://reviews.llvm.org/D12414#375657</a>, @jingyue wrote:<br>

<br>

> It doesn't work with the new alias analysis infrastructure.<br>

><br>

> I didn't manage to find a way to run it with `opt`. The new AliasAnalysis needs to know and populate all potential alias analyses in a target-independent phase. However, NVPTXAliasAnalysis is target-dependent and not even visible to AliasAnalysis (e.g. compiling LLVM with NVPTX disabled).<br>

<br>

<br>

</span>Do you have an alternative plan?<br>

<br>

It seems like we could add a feature in the new AA to query a target-provided analysis.<br>

<br>

<br>

<a href="http://reviews.llvm.org/D12414" rel="noreferrer" target="_blank">http://reviews.llvm.org/D12414</a><br>

<br>

<br>

<br>

</blockquote></div><br></div>

</blockquote><br><br><br>-- <br><div><span name="x"></span>Hal Finkel<br>Assistant Computational Scientist<br>Leadership Computing Facility<br>Argonne National Laboratory<span name="x"></span><br></div></div></body></html>