[all-commits] [llvm/llvm-project] 191e46: [AA] Move Depth member from AAResults to AAQI (NFC)

Nikita Popov via All-commits all-commits at lists.llvm.org
Fri Feb 12 12:43:01 PST 2021


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: 191e469edee619f578076d775ce1ca19c9ecc103
      https://github.com/llvm/llvm-project/commit/191e469edee619f578076d775ce1ca19c9ecc103
  Author: Nikita Popov <nikita.ppv at gmail.com>
  Date:   2021-02-12 (Fri, 12 Feb 2021)

  Changed paths:
    M llvm/include/llvm/Analysis/AliasAnalysis.h
    M llvm/lib/Analysis/AliasAnalysis.cpp
    M llvm/lib/Analysis/BasicAliasAnalysis.cpp

  Log Message:
  -----------
  [AA] Move Depth member from AAResults to AAQI (NFC)

Rather than storing the query depth in AAResults, store it in AAQI.
This makes more sense, as it is a property of the query. This
sidesteps the issue of D94363, fixing slightly inaccurate AA
statistics. Additionally, I plan to use the Depth from BasicAA in
the future, where fetching it from AAResults would be unreliable.

This change is not quite as straightforward as it seems, because
we need to preserve the depth when creating a new AAQI for recursive
queries across phis. I'm adding a new method for this, as we may
need to preserve additional information here in the future.




More information about the All-commits mailing list