[llvm] [DirectX] Infrastructure to collect shader flags for each function (PR #112967)
Chris B via llvm-commits
llvm-commits at lists.llvm.org
Mon Nov 18 13:39:23 PST 2024
================
@@ -120,16 +127,17 @@ class ShaderFlagsAnalysisPrinter
/// This is required because the passes that will depend on this are codegen
/// passes which run through the legacy pass manager.
class ShaderFlagsAnalysisWrapper : public ModulePass {
- DXILModuleShaderFlagsInfo MSFI;
+ std::unique_ptr<DXILModuleShaderFlagsInfo> MSFI;
----------------
llvm-beanz wrote:
+1
I also question if we need the optional. Why was the old pattern of the uninitialized flag structure a problem?
https://github.com/llvm/llvm-project/pull/112967
More information about the llvm-commits
mailing list