barcisz added inline comments.
================ Comment at: clang-tools-extra/clang-tidy/cuda/CudaTidyModule.cpp:25 + +// Register the GoogleTidyModule using this statically initialized variable. +static ClangTidyModuleRegistry::Add<CudaModule> ---------------- tschuett wrote: > Is Google a copy and paste error? Done ================ Comment at: clang-tools-extra/clang-tidy/cuda/CudaTidyModule.cpp:32 +// This anchor is used to force the linker to link in the generated object file +// and thus register the GoogleModule. +volatile int CudaModuleAnchorSource = 0; ---------------- tschuett wrote: > Is Google a copy and paste error? Done ================ Comment at: clang-tools-extra/test/lit.cfg.py:19 # suffixes: A list of file extensions to treat as test files. -config.suffixes = ['.c', '.cpp', '.hpp', '.m', '.mm', '.cu', '.ll', '.cl', '.s', +config.suffixes = ['.c', '.cpp', '.cu', '.hpp', '.m', '.mm', '.cu', '.ll', '.cl', '.s', '.modularize', '.module-map-checker', '.test'] ---------------- ivanmurashko wrote: > `.cu` is specified several times there. It looks like the change is not > necessary at the line Done Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D133436/new/ https://reviews.llvm.org/D133436 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
