================
@@ -297,7 +300,16 @@ class Checker {
   // step-function slowdowns due to CPU scaling.
   // We take the median of 5 measurements, and after every check discard the
   // measurement if the baseline changed by >3%.
-  void checkTidyTimes() {
+  bool checkTidyTimes() {
+    // Keep all other modules active while varying only tidy's configuration.
+    auto *TidyModule =
+        Inputs.FeatureModules
+            ? Inputs.FeatureModules->get<ClangTidyFeatureModule>()
----------------
JVApen wrote:

Just thinking out loud here: should the checkTiming be a function on the tidy 
module? Should the module interface have a 'afterBuildAst' in their the 
interface or it's AST listener? 

https://github.com/llvm/llvm-project/pull/236054
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to