Skip to content

Commit 777e5cd

Browse files
committed
[OpenMP][Fix] Check Expected<T> and assert if fail
This checks the Expected for value. This should probably not fail, and if it does, I think we can safely abort with a failing assert. Change-Id: Ia017e1b8e321d8d5f8363d5d76642544f8730405
1 parent 027d1b1 commit 777e5cd

File tree

1 file changed

+4
-0
lines changed

1 file changed

+4
-0
lines changed

openmp/libomptarget/plugins-nextgen/amdgpu/utils/UtilitiesRTL.h

Lines changed: 4 additions & 0 deletions
Original file line numberDiff line numberDiff line change
@@ -140,6 +140,10 @@ extractXnackModeFromBinary(const __tgt_device_image *TgtImage) {
140140
Expected<ELF64LEObjectFile> ElfOrErr =
141141
ELF64LEObjectFile::create(MemoryBufferRef(Buffer, /*Identifier=*/""),
142142
/*InitContent=*/false);
143+
if (!ElfOrErr) {
144+
consumeError(ElfOrErr.takeError());
145+
return XNACK_UNSUPPORTED;
146+
}
143147

144148
u_int16_t EFlags = ElfOrErr->getELFFile().getHeader().e_flags;
145149
unsigned XnackFlags = EFlags & ELF::EF_AMDGPU_FEATURE_XNACK_V4;

0 commit comments

Comments
 (0)