[PATCH] D94363: [AA] Pass AAResults through AAQueryInfo

[Nikita Popov via Phabricator via llvm-commits]

nikic added a comment.

In D94363#2499809 <https://reviews.llvm.org/D94363#2499809>, @asbirlea wrote:

> Do you have an idea if there is overhead of adding the AAR object to each query (each query info object)? (vs the `setAAResults` being done once for all the AAs available).

It looks like adding AAR to the query doesn't have noticeable overhead: https://llvm-compile-time-tracker.com/compare.php?from=6ee44ee7c3cee47cb070ccb961359df8de252f36&to=9787880b8fe7ed5f62f15eaeed561a6ce09fdfd6&stat=instructions

There is however a small cost to passing AAQI through getModRefBehavior: https://llvm-compile-time-tracker.com/compare.php?from=33be50daa9ce1074c3b423a4ab27c70c0722113a&to=6ee44ee7c3cee47cb070ccb961359df8de252f36&stat=instructions


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D94363/new/

https://reviews.llvm.org/D94363