Skip to content

Commit 7d1d83d

Browse files
bwlodarczsys-ce-bb
authored andcommitted
Memory leak fix in LLVMToSPIRVBase::transExecutionMode (#2269)
Allocated SPIRVExecutionMode class wasn't added as dependant resource in SPIRVModule. Fix some ASan errors in issue #2233. Original commit: KhronosGroup/SPIRV-LLVM-Translator@1a19f6f
1 parent 70ec903 commit 7d1d83d

File tree

1 file changed

+2
-2
lines changed

1 file changed

+2
-2
lines changed

llvm-spirv/lib/SPIRV/SPIRVWriter.cpp

Lines changed: 2 additions & 2 deletions
Original file line numberDiff line numberDiff line change
@@ -5735,8 +5735,8 @@ bool LLVMToSPIRVBase::transExecutionMode() {
57355735
break;
57365736
unsigned NBarrierCnt = 0;
57375737
N.get(NBarrierCnt);
5738-
BF->addExecutionMode(new SPIRVExecutionMode(
5739-
BF, static_cast<ExecutionMode>(EMode), NBarrierCnt));
5738+
BF->addExecutionMode(BM->add(new SPIRVExecutionMode(
5739+
BF, static_cast<ExecutionMode>(EMode), NBarrierCnt)));
57405740
BM->addExtension(ExtensionID::SPV_INTEL_vector_compute);
57415741
BM->addCapability(CapabilityVectorComputeINTEL);
57425742
} break;

0 commit comments

Comments
 (0)