[llvm] [Analysis] Remove global state from `PluginInline{Advisor,Order}Analysis`. (PR #114615)

Arthur Eubanks via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 15 14:04:31 PST 2024


================
@@ -456,13 +461,6 @@ template <typename IRUnitT, typename... ExtraArgTs> class AnalysisManager {
   /// argument will not be called and this function will return false.
   /// Otherwise, we register the analysis returned by calling \c PassBuilder(),
   /// and this function returns true.
-  ///
-  /// (Note: Although the return value of this function indicates whether or not
-  /// an analysis was previously registered, there intentionally isn't a way to
-  /// query this directly.  Instead, you should just register all the analyses
----------------
aeubanks wrote:

Something like "Although the return value of this function indicates whether or not
  /// an analysis was previously registered, you should just register all the analyses..."

https://github.com/llvm/llvm-project/pull/114615


More information about the llvm-commits mailing list