Clang-format and makefile fix, and support no RTTI.

Author: mcourteaux

Makefile

@@ -483,6 +483,7 @@ SOURCE_FILES = \
   CodeGen_WebAssembly.cpp \
   CodeGen_WebGPU_Dev.cpp \
   CodeGen_X86.cpp \
+  CompilerProfiling.cpp \
   CompilerLogger.cpp \
   ConstantBounds.cpp \
   ConstantInterval.cpp \
@@ -557,7 +558,6 @@ SOURCE_FILES = \
   Pipeline.cpp \
   Prefetch.cpp \
   PrintLoopNest.cpp \
-  ProfiledIRVisitor.cpp \
   Profiling.cpp \
   PurifyIndexMath.cpp \
   PythonExtensionGen.cpp \
@@ -678,6 +678,7 @@ HEADER_FILES = \
   CodeGen_PyTorch.h \
   CodeGen_Targets.h \
   CodeGen_WebGPU_Dev.h \
+  CompilerProfiling.h \
   CompilerLogger.h \
   ConciseCasts.h \
   CPlusPlusMangle.h \

src/AddParameterChecks.cpp

@@ -1,9 +1,9 @@
 #include "AddParameterChecks.h"
+#include "CompilerProfiling.h"
 #include "IROperator.h"
 #include "IRVisitor.h"
 #include "Substitute.h"
 #include "Target.h"
-#include "CompilerProfiling.h"
 
 namespace Halide {
 namespace Internal {

src/CodeGen_LLVM.cpp

@@ -9,6 +9,7 @@
 #include "CodeGen_Posix.h"
 #include "CodeGen_Targets.h"
 #include "CompilerLogger.h"
+#include "CompilerProfiling.h"
 #include "Debug.h"
 #include "Deinterleave.h"
 #include "EmulateFloat16Math.h"
@@ -28,7 +29,6 @@
 #include "Simplify.h"
 #include "StrictifyFloat.h"
 #include "Util.h"
-#include "CompilerProfiling.h"
 
 // MSVC won't set __cplusplus correctly unless certain compiler flags are set
 // (and CMake doesn't set those flags for you even if you specify C++17),

src/CompilerProfiling.h

@@ -184,7 +184,11 @@ template<typename Base>
 class Profiled : public Base {
 public:
     using Base::Base;
+#ifdef HALIDE_ENABLE_RTTI
     const char *tag = typeid(Base).name();
+#else
+    const char *tag = "Unknown (no RTTI)";
+#endif
 
 #define PROFILE_VISIT_STMT_OVERRIDE(T)                                    \
     auto visit(const T *op) -> decltype(this->Base::visit(op)) override { \
@@ -218,9 +222,11 @@ using Profiled = Profiling::Profiled<Base>;
 #else
 
 namespace Profiling {
-inline void generic_zone_begin(const char *src_tag, unsigned data = 0) {}
-inline void generic_zone_end(const char *src_tag, unsigned data = 0) {}
+inline void generic_zone_begin(const char *src_tag, unsigned data = 0) {
 }
+inline void generic_zone_end(const char *src_tag, unsigned data = 0) {
+}
+}  // namespace Profiling
 
 template<typename Base>
 using Profiled = Base;

src/Expr.cpp

@@ -6,7 +6,7 @@ namespace Internal {
 
 const char *IRNodeType_string(IRNodeType type) {
     switch (type) {
-#define PROFILE_NODE_CASE(T)                                    \
+#define PROFILE_NODE_CASE(T)              \
     case Halide::Internal::IRNodeType::T: \
         return #T;
 

src/IRVisitor.h

@@ -168,7 +168,13 @@ class IRGraphVisitor : public IRVisitor {
 template<typename T, typename ExprRet, typename StmtRet>
 class VariadicVisitor {
 private:
+#ifdef WITH_COMPILER_PROFILING
+#ifdef HALIDE_ENABLE_RTTI
     const char *name = typeid(T).name();
+#else
+    const char *name = "VariadicVisitor";
+#endif
+#endif
 
     template<typename... Args>
     ExprRet dispatch_expr(const BaseExprNode *node, Args &&...args) {

src/LLVM_Runtime_Linker.cpp

@@ -1,8 +1,8 @@
 #include "LLVM_Runtime_Linker.h"
+#include "CompilerProfiling.h"
 #include "Error.h"
 #include "LLVM_Headers.h"
 #include "Target.h"
-#include "CompilerProfiling.h"
 
 namespace Halide {
 

src/Pipeline.cpp

@@ -4,6 +4,7 @@
 #include "Argument.h"
 #include "Callable.h"
 #include "CodeGen_Internal.h"
+#include "CompilerProfiling.h"
 #include "Deserialization.h"
 #include "FindCalls.h"
 #include "Func.h"
@@ -17,7 +18,6 @@
 #include "RealizationOrder.h"
 #include "Serialization.h"
 #include "WasmExecutor.h"
-#include "CompilerProfiling.h"
 
 using namespace Halide::Internal;
 

src/Prefetch.cpp

@@ -1,6 +1,6 @@
 #include <map>
-#include <unordered_map>
 #include <string>
+#include <unordered_map>
 #include <utility>
 
 #include "Bounds.h"
@@ -18,9 +18,9 @@ namespace Halide {
 namespace Internal {
 
 using std::map;
-using std::unordered_map;
 using std::set;
 using std::string;
+using std::unordered_map;
 using std::vector;
 
 /**

src/Simplify.cpp

@@ -3,9 +3,9 @@
 
 #include "CSE.h"
 #include "CompilerLogger.h"
+#include "CompilerProfiling.h"
 #include "IRMutator.h"
 #include "Substitute.h"
-#include "CompilerProfiling.h"
 
 namespace Halide {
 namespace Internal {