diff --git a/llvm/lib/Target/DirectX/DXILPrepare.cpp b/llvm/lib/Target/DirectX/DXILPrepare.cpp index 889de3a81536d..56098864e987f 100644 --- a/llvm/lib/Target/DirectX/DXILPrepare.cpp +++ b/llvm/lib/Target/DirectX/DXILPrepare.cpp @@ -246,7 +246,7 @@ class DXILPrepareModule : public ModulePass { DXILPrepareModule() : ModulePass(ID) {} void getAnalysisUsage(AnalysisUsage &AU) const override { AU.addPreserved(); - AU.addPreserved(); + AU.addPreserved(); } static char ID; // Pass identification. }; diff --git a/llvm/lib/Target/DirectX/DXILPrettyPrinter.cpp b/llvm/lib/Target/DirectX/DXILPrettyPrinter.cpp index 7ae568c5ae53a..99cc4067b1d62 100644 --- a/llvm/lib/Target/DirectX/DXILPrettyPrinter.cpp +++ b/llvm/lib/Target/DirectX/DXILPrettyPrinter.cpp @@ -41,7 +41,7 @@ class DXILPrettyPrinter : public llvm::ModulePass { bool runOnModule(Module &M) override; void getAnalysisUsage(AnalysisUsage &AU) const override { AU.setPreservesAll(); - AU.addRequired(); + AU.addRequired(); } }; } // namespace @@ -49,12 +49,12 @@ class DXILPrettyPrinter : public llvm::ModulePass { char DXILPrettyPrinter::ID = 0; INITIALIZE_PASS_BEGIN(DXILPrettyPrinter, "dxil-pretty-printer", "DXIL Metadata Pretty Printer", true, true) -INITIALIZE_PASS_DEPENDENCY(DXILResourceWrapper) +INITIALIZE_PASS_DEPENDENCY(DXILResourceMDWrapper) INITIALIZE_PASS_END(DXILPrettyPrinter, "dxil-pretty-printer", "DXIL Metadata Pretty Printer", true, true) bool DXILPrettyPrinter::runOnModule(Module &M) { - dxil::Resources &Res = getAnalysis().getDXILResource(); + dxil::Resources &Res = getAnalysis().getDXILResource(); Res.print(OS); return false; } diff --git a/llvm/lib/Target/DirectX/DXILResourceAnalysis.cpp b/llvm/lib/Target/DirectX/DXILResourceAnalysis.cpp index 0b2f0d827ebbc..33e0119807bb8 100644 --- a/llvm/lib/Target/DirectX/DXILResourceAnalysis.cpp +++ b/llvm/lib/Target/DirectX/DXILResourceAnalysis.cpp @@ -18,35 +18,35 @@ using namespace llvm; #define DEBUG_TYPE "dxil-resource-analysis" -dxil::Resources DXILResourceAnalysis::run(Module &M, - ModuleAnalysisManager &AM) { +dxil::Resources DXILResourceMDAnalysis::run(Module &M, + ModuleAnalysisManager &AM) { dxil::Resources R; R.collect(M); return R; } -AnalysisKey DXILResourceAnalysis::Key; +AnalysisKey DXILResourceMDAnalysis::Key; -PreservedAnalyses DXILResourcePrinterPass::run(Module &M, - ModuleAnalysisManager &AM) { - dxil::Resources Res = AM.getResult(M); +PreservedAnalyses DXILResourceMDPrinterPass::run(Module &M, + ModuleAnalysisManager &AM) { + dxil::Resources Res = AM.getResult(M); Res.print(OS); return PreservedAnalyses::all(); } -char DXILResourceWrapper::ID = 0; -INITIALIZE_PASS_BEGIN(DXILResourceWrapper, DEBUG_TYPE, +char DXILResourceMDWrapper::ID = 0; +INITIALIZE_PASS_BEGIN(DXILResourceMDWrapper, DEBUG_TYPE, "DXIL resource Information", true, true) -INITIALIZE_PASS_END(DXILResourceWrapper, DEBUG_TYPE, +INITIALIZE_PASS_END(DXILResourceMDWrapper, DEBUG_TYPE, "DXIL resource Information", true, true) -bool DXILResourceWrapper::runOnModule(Module &M) { +bool DXILResourceMDWrapper::runOnModule(Module &M) { Resources.collect(M); return false; } -DXILResourceWrapper::DXILResourceWrapper() : ModulePass(ID) {} +DXILResourceMDWrapper::DXILResourceMDWrapper() : ModulePass(ID) {} -void DXILResourceWrapper::print(raw_ostream &OS, const Module *) const { +void DXILResourceMDWrapper::print(raw_ostream &OS, const Module *) const { Resources.print(OS); } diff --git a/llvm/lib/Target/DirectX/DXILResourceAnalysis.h b/llvm/lib/Target/DirectX/DXILResourceAnalysis.h index bce41160b95ec..3a2b8a9fd39d5 100644 --- a/llvm/lib/Target/DirectX/DXILResourceAnalysis.h +++ b/llvm/lib/Target/DirectX/DXILResourceAnalysis.h @@ -20,8 +20,9 @@ namespace llvm { /// Analysis pass that exposes the \c DXILResource for a module. -class DXILResourceAnalysis : public AnalysisInfoMixin { - friend AnalysisInfoMixin; +class DXILResourceMDAnalysis + : public AnalysisInfoMixin { + friend AnalysisInfoMixin; static AnalysisKey Key; public: @@ -29,25 +30,26 @@ class DXILResourceAnalysis : public AnalysisInfoMixin { dxil::Resources run(Module &M, ModuleAnalysisManager &AM); }; -/// Printer pass for the \c DXILResourceAnalysis results. -class DXILResourcePrinterPass : public PassInfoMixin { +/// Printer pass for the \c DXILResourceMDAnalysis results. +class DXILResourceMDPrinterPass + : public PassInfoMixin { raw_ostream &OS; public: - explicit DXILResourcePrinterPass(raw_ostream &OS) : OS(OS) {} + explicit DXILResourceMDPrinterPass(raw_ostream &OS) : OS(OS) {} PreservedAnalyses run(Module &M, ModuleAnalysisManager &AM); static bool isRequired() { return true; } }; /// The legacy pass manager's analysis pass to compute DXIL resource /// information. -class DXILResourceWrapper : public ModulePass { +class DXILResourceMDWrapper : public ModulePass { dxil::Resources Resources; public: static char ID; // Pass identification, replacement for typeid - DXILResourceWrapper(); + DXILResourceMDWrapper(); dxil::Resources &getDXILResource() { return Resources; } const dxil::Resources &getDXILResource() const { return Resources; } diff --git a/llvm/lib/Target/DirectX/DXILTranslateMetadata.cpp b/llvm/lib/Target/DirectX/DXILTranslateMetadata.cpp index ae6d6f96904c8..583bce0f50e70 100644 --- a/llvm/lib/Target/DirectX/DXILTranslateMetadata.cpp +++ b/llvm/lib/Target/DirectX/DXILTranslateMetadata.cpp @@ -33,7 +33,7 @@ class DXILTranslateMetadata : public ModulePass { void getAnalysisUsage(AnalysisUsage &AU) const override { AU.setPreservesAll(); - AU.addRequired(); + AU.addRequired(); AU.addRequired(); } @@ -51,7 +51,7 @@ bool DXILTranslateMetadata::runOnModule(Module &M) { dxil::createDXILVersionMD(M); const dxil::Resources &Res = - getAnalysis().getDXILResource(); + getAnalysis().getDXILResource(); Res.write(M); const uint64_t Flags = static_cast( @@ -69,7 +69,7 @@ ModulePass *llvm::createDXILTranslateMetadataPass() { INITIALIZE_PASS_BEGIN(DXILTranslateMetadata, "dxil-metadata-emit", "DXIL Metadata Emit", false, false) -INITIALIZE_PASS_DEPENDENCY(DXILResourceWrapper) +INITIALIZE_PASS_DEPENDENCY(DXILResourceMDWrapper) INITIALIZE_PASS_DEPENDENCY(ShaderFlagsAnalysisWrapper) INITIALIZE_PASS_END(DXILTranslateMetadata, "dxil-metadata-emit", "DXIL Metadata Emit", false, false) diff --git a/llvm/lib/Target/DirectX/DirectX.h b/llvm/lib/Target/DirectX/DirectX.h index 11b5412c21d78..d056ae2bc488e 100644 --- a/llvm/lib/Target/DirectX/DirectX.h +++ b/llvm/lib/Target/DirectX/DirectX.h @@ -47,7 +47,7 @@ void initializeDXILTranslateMetadataPass(PassRegistry &); ModulePass *createDXILTranslateMetadataPass(); /// Initializer for DXILTranslateMetadata. -void initializeDXILResourceWrapperPass(PassRegistry &); +void initializeDXILResourceMDWrapperPass(PassRegistry &); /// Pass to pretty print DXIL metadata. ModulePass *createDXILPrettyPrinterPass(raw_ostream &OS); diff --git a/llvm/lib/Target/DirectX/DirectXPassRegistry.def b/llvm/lib/Target/DirectX/DirectXPassRegistry.def index 1b326d020d511..7544172ab94e4 100644 --- a/llvm/lib/Target/DirectX/DirectXPassRegistry.def +++ b/llvm/lib/Target/DirectX/DirectXPassRegistry.def @@ -17,7 +17,7 @@ #define MODULE_ANALYSIS(NAME, CREATE_PASS) #endif MODULE_ANALYSIS("dx-shader-flags", dxil::ShaderFlagsAnalysis()) -MODULE_ANALYSIS("dxil-resource", DXILResourceAnalysis()) +MODULE_ANALYSIS("dxil-resource-md", DXILResourceMDAnalysis()) #undef MODULE_ANALYSIS #ifndef MODULE_PASS @@ -25,5 +25,5 @@ MODULE_ANALYSIS("dxil-resource", DXILResourceAnalysis()) #endif // TODO: rename to print after NPM switch MODULE_PASS("print-dx-shader-flags", dxil::ShaderFlagsAnalysisPrinter(dbgs())) -MODULE_PASS("print-dxil-resource", DXILResourcePrinterPass(dbgs())) +MODULE_PASS("print-dxil-resource-md", DXILResourceMDPrinterPass(dbgs())) #undef MODULE_PASS diff --git a/llvm/lib/Target/DirectX/DirectXTargetMachine.cpp b/llvm/lib/Target/DirectX/DirectXTargetMachine.cpp index e6dbb25b710ec..92bd69b69684f 100644 --- a/llvm/lib/Target/DirectX/DirectXTargetMachine.cpp +++ b/llvm/lib/Target/DirectX/DirectXTargetMachine.cpp @@ -46,7 +46,7 @@ extern "C" LLVM_EXTERNAL_VISIBILITY void LLVMInitializeDirectXTarget() { initializeDXContainerGlobalsPass(*PR); initializeDXILOpLoweringLegacyPass(*PR); initializeDXILTranslateMetadataPass(*PR); - initializeDXILResourceWrapperPass(*PR); + initializeDXILResourceMDWrapperPass(*PR); initializeShaderFlagsAnalysisWrapperPass(*PR); } diff --git a/llvm/test/CodeGen/DirectX/UAVMetadata.ll b/llvm/test/CodeGen/DirectX/UAVMetadata.ll index 0bc8a8cfcd713..bdad9fd40c9bd 100644 --- a/llvm/test/CodeGen/DirectX/UAVMetadata.ll +++ b/llvm/test/CodeGen/DirectX/UAVMetadata.ll @@ -1,5 +1,5 @@ ; RUN: opt -S -dxil-metadata-emit < %s | FileCheck %s -; RUN: opt -S --passes="print-dxil-resource" < %s 2>&1 | FileCheck %s --check-prefix=PRINT +; RUN: opt -S --passes="print-dxil-resource-md" < %s 2>&1 | FileCheck %s --check-prefix=PRINT ; RUN: llc %s --filetype=asm -o - < %s 2>&1 | FileCheck %s --check-prefixes=CHECK,PRINT target datalayout = "e-m:e-p:32:32-i1:32-i8:8-i16:16-i32:32-i64:64-f16:16-f32:32-f64:64-n8:16:32:64" diff --git a/llvm/test/CodeGen/DirectX/cbuf.ll b/llvm/test/CodeGen/DirectX/cbuf.ll index d07cc1e880b1a..38f08fad995d1 100644 --- a/llvm/test/CodeGen/DirectX/cbuf.ll +++ b/llvm/test/CodeGen/DirectX/cbuf.ll @@ -1,5 +1,5 @@ ; RUN: opt -S -dxil-metadata-emit < %s | FileCheck %s --check-prefix=DXILMD -; RUN: opt -S --passes="print-dxil-resource" < %s 2>&1 | FileCheck %s --check-prefix=PRINT +; RUN: opt -S --passes="print-dxil-resource-md" < %s 2>&1 | FileCheck %s --check-prefix=PRINT target datalayout = "e-m:e-p:32:32-i1:32-i8:8-i16:16-i32:32-i64:64-f16:16-f32:32-f64:64-n8:16:32:64" target triple = "dxil-unknown-shadermodel6.7-library"