[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
> Do you have an alternative plan?
> It seems like we could add a feature in the new AA to query a
> target-provided analysis.
-------------- next part --------------
An HTML attachment was scrubbed...
More information about the llvm-commits