[all-commits] [llvm/llvm-project] a8a33b: [NFC][SPIRV] Misc code cleanup in SPIRV Target (#1...

Rahul Joshi via All-commits all-commits at lists.llvm.org
Tue Apr 1 08:35:27 PDT 2025


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: a8a33bab69ccdeca9b0e1e6730ad17230431d2ab
      https://github.com/llvm/llvm-project/commit/a8a33bab69ccdeca9b0e1e6730ad17230431d2ab
  Author: Rahul Joshi <rjoshi at nvidia.com>
  Date:   2025-04-01 (Tue, 01 Apr 2025)

  Changed paths:
    M llvm/lib/Target/SPIRV/Analysis/SPIRVConvergenceRegionAnalysis.cpp
    M llvm/lib/Target/SPIRV/Analysis/SPIRVConvergenceRegionAnalysis.h
    M llvm/lib/Target/SPIRV/SPIRVBuiltins.cpp
    M llvm/lib/Target/SPIRV/SPIRVCommandLine.cpp
    M llvm/lib/Target/SPIRV/SPIRVCommandLine.h
    M llvm/lib/Target/SPIRV/SPIRVEmitIntrinsics.cpp
    M llvm/lib/Target/SPIRV/SPIRVEmitNonSemanticDI.cpp
    M llvm/lib/Target/SPIRV/SPIRVGlobalRegistry.cpp
    M llvm/lib/Target/SPIRV/SPIRVStructurizer.cpp
    M llvm/lib/Target/SPIRV/SPIRVStructurizerWrapper.h
    M llvm/unittests/Target/SPIRV/SPIRVConvergenceRegionAnalysisTests.cpp

  Log Message:
  -----------
  [NFC][SPIRV] Misc code cleanup in SPIRV Target (#133764)

- Use static instead of anonymous namespace for file local functions.
- Enclose file-local classes in anonymous namespace.
- Eliminate `llvm::` qualifier when file has `using namespace llvm`.
- Eliminate namespace surrounding entire code in
SPIRVConvergenceRegionAnalysis.cpp file.
- Eliminate call to `initializeSPIRVStructurizerPass` from the pass
constructor (https://github.com/llvm/llvm-project/issues/111767)



To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications


More information about the All-commits mailing list