[llvm] [DXIL][Analysis] Add DXILMetadataAnalysis pass (PR #102079)
S. Bharadwaj Yadavalli via llvm-commits
llvm-commits at lists.llvm.org
Thu Aug 8 15:32:09 PDT 2024
================
@@ -0,0 +1,85 @@
+//=- DXILMetadataAnalysis.h - Representation of Module metadata --*- C++ -*-=//
+//
+// Part of the LLVM Project, under the Apache License v2.0 with LLVM Exceptions.
+// See https://llvm.org/LICENSE.txt for license information.
+// SPDX-License-Identifier: Apache-2.0 WITH LLVM-exception
+//
+//===----------------------------------------------------------------------===//
+
+#ifndef LLVM_ANALYSIS_DXILMETADATA_H
+#define LLVM_ANALYSIS_DXILMETADATA_H
+
+#include "llvm/ADT/MapVector.h"
+#include "llvm/IR/PassManager.h"
+#include "llvm/IR/Value.h"
+#include "llvm/Pass.h"
+#include "llvm/Support/DXILABI.h"
+#include "llvm/Support/VersionTuple.h"
+#include "llvm/TargetParser/Triple.h"
+
+namespace llvm {
+
+namespace dxil {
+
+struct ModuleMetadataInfo {
+ VersionTuple DXILVersion{};
+ VersionTuple ValidatorVersion{};
+ VersionTuple ShaderModelVersion{};
+ Triple::EnvironmentType ShaderStage = Triple::UnknownEnvironment;
+
+ void init(Module &);
----------------
bharadwajy wrote:
> Rather than having an `init` method in the public interface like this it's better to construct the analysis result in a `static` function or a helper class in an anonymous namespace in the `.cpp` file. That way we don't need to leak implementation details to the header.
Moved the initialization into a constructor in `DXILMetadataAnalysis.cpp`.
https://github.com/llvm/llvm-project/pull/102079
More information about the llvm-commits
mailing list