Skip to content

Commit 881dd36

Browse files
author
joaosaffran
committed
address comments
1 parent ea54904 commit 881dd36

File tree

2 files changed

+3
-4
lines changed

2 files changed

+3
-4
lines changed

llvm/lib/Target/DirectX/DXContainerGlobals.cpp

Lines changed: 2 additions & 1 deletion
Original file line numberDiff line numberDiff line change
@@ -162,7 +162,8 @@ void DXContainerGlobals::addRootSignature(Module &M,
162162

163163
auto &RSA = getAnalysis<RootSignatureAnalysisWrapper>().getRSInfo();
164164
const Function *EntryFunction = MMI.EntryPropertyVec[0].Entry;
165-
const auto &RS = RSA.getDescForFunction(EntryFunction);
165+
const std::optional<mcdxbc::RootSignatureDesc> &RS =
166+
RSA.getDescForFunction(EntryFunction);
166167

167168
if (!RS)
168169
return;

llvm/lib/Target/DirectX/DXILRootSignature.h

Lines changed: 1 addition & 3 deletions
Original file line numberDiff line numberDiff line change
@@ -47,7 +47,7 @@ class RootSignatureBindingInfo {
4747
RootSignatureBindingInfo() = default;
4848
RootSignatureBindingInfo(
4949
SmallDenseMap<const Function *, mcdxbc::RootSignatureDesc> Map)
50-
: FuncToRsMap(Map) {};
50+
: FuncToRsMap(Map){};
5151

5252
iterator find(const Function *F) { return FuncToRsMap.find(F); }
5353

@@ -88,8 +88,6 @@ class RootSignatureAnalysisWrapper : public ModulePass {
8888

8989
public:
9090
static char ID;
91-
using Result = RootSignatureBindingInfo;
92-
9391
RootSignatureAnalysisWrapper() : ModulePass(ID) {}
9492

9593
RootSignatureBindingInfo &getRSInfo() { return *FuncToRsMap; }

0 commit comments

Comments
 (0)