[PATCH] D12414: [NVPTX] add an NVPTX-specific alias analysis

[Daniel Berlin via llvm-commits]

+1.
If we need this, we should make it work ;)

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
(if they did, they didn't discuss it on llvm-dev)


On Tue, Mar 15, 2016 at 10:57 AM, Hal Finkel <hfinkel at anl.gov> wrote:

> hfinkel added a comment.
>
> In http://reviews.llvm.org/D12414#375657, @jingyue wrote:
>
> > It doesn't work with the new alias analysis infrastructure.
> >
> > 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).
>
>
> Do you have an alternative plan?
>
> It seems like we could add a feature in the new AA to query a
> target-provided analysis.
>
>
> http://reviews.llvm.org/D12414
>
>
>
>
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20160315/1f58c473/attachment.html>