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

Daniel Berlin via llvm-commits llvm-commits at lists.llvm.org
Tue Mar 15 11:03:35 PDT 2016

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>

More information about the llvm-commits mailing list