Skip to content

Commit e7d96da

Browse files
authored
Fix SPIRVOps initialization (#4174)
SPIRV::TranslatorOpts TransOpt{SPIRV::VersionNumber::SPIRV_1_4} was unused.
1 parent ceef901 commit e7d96da

File tree

1 file changed

+3
-4
lines changed

1 file changed

+3
-4
lines changed

third_party/intel/lib/Target/SPIRV/SPIRVTranslation.cpp

+3-4
Original file line numberDiff line numberDiff line change
@@ -105,8 +105,8 @@ class SmallVectorBuffer : public std::streambuf {
105105
SmallVectorBuffer(llvm::SmallVectorImpl<char> &O) : OS(O) {}
106106
};
107107

108-
static SPIRV::TranslatorOpts getSPIRVOopts() {
109-
SPIRV::TranslatorOpts SPIRVOpts;
108+
static SPIRV::TranslatorOpts getSPIRVOpts() {
109+
SPIRV::TranslatorOpts SPIRVOpts{SPIRV::VersionNumber::SPIRV_1_4};
110110
static constexpr std::array<SPIRV::ExtensionID, 16> AllowedExtensions{
111111
SPIRV::ExtensionID::SPV_EXT_shader_atomic_float_add,
112112
SPIRV::ExtensionID::SPV_INTEL_arbitrary_precision_integers,
@@ -129,7 +129,6 @@ static SPIRV::TranslatorOpts getSPIRVOopts() {
129129
SPIRVOpts.setPreserveOCLKernelArgTypeMetadataThroughString(true);
130130
SPIRVOpts.setPreserveAuxData(false);
131131
SPIRVOpts.setSPIRVAllowUnknownIntrinsics({"llvm.genx.GenISA."});
132-
SPIRV::TranslatorOpts TransOpt{SPIRV::VersionNumber::SPIRV_1_4};
133132

134133
for (auto &Ext : AllowedExtensions)
135134
SPIRVOpts.setAllowedToUseExtension(Ext, true);
@@ -156,7 +155,7 @@ std::string translateLLVMIRToSPIRV(llvm::Module &module) {
156155
std::ostream OS(&StreamBuf);
157156
std::string Err;
158157

159-
SPIRV::TranslatorOpts SPIRVOpts = getSPIRVOopts();
158+
SPIRV::TranslatorOpts SPIRVOpts = getSPIRVOpts();
160159

161160
#if defined(LLVM_SPIRV_BACKEND_TARGET_PRESENT)
162161
int SpvTranslateMode = 0;

0 commit comments

Comments
 (0)