Skip to content

Commit 3af4447

Browse files
shiltianronlieb
authored andcommitted
[AMDGPU][Comgr] Fix test issues in amd/comgr/test/unbundle_hip_test.c
1 parent 58a19f3 commit 3af4447

File tree

1 file changed

+10
-12
lines changed

1 file changed

+10
-12
lines changed

amd/comgr/test/unbundle_hip_test.c

Lines changed: 10 additions & 12 deletions
Original file line numberDiff line numberDiff line change
@@ -182,11 +182,10 @@ int main(int Argc, char *Argv[]) {
182182
Status = amd_comgr_release_data(DataElement);
183183
checkError(Status, "amd_comgr_release_data");
184184

185-
// TODO: Re-enable after finalizing LLVM PR #122629
186-
// if (BytesSize != 0) {
187-
// printf("Bitcode host element size: %ld (expected 0)\n", BytesSize);
188-
// exit(1);
189-
// }
185+
if (!BytesSize) {
186+
printf("Bitcode host empty (expected non-empty)\n");
187+
exit(1);
188+
}
190189

191190
// bitcode hip-gfx900 element (non-empty)
192191
Status = amd_comgr_action_data_get_data(
@@ -248,11 +247,10 @@ int main(int Argc, char *Argv[]) {
248247
Status = amd_comgr_release_data(DataElement);
249248
checkError(Status, "amd_comgr_release_data");
250249

251-
// TODO: Re-enable after finalizing LLVM PR #122629
252-
// if (BytesSize != 0) {
253-
// printf("Object host element size: %ld (expected empty)\n", BytesSize);
254-
// exit(1);
255-
//}
250+
if (BytesSize != 0) {
251+
printf("Object host element size: %ld (expected empty)\n", BytesSize);
252+
exit(1);
253+
}
256254

257255
// object hip-gfx900 element (non-empty)
258256
Status = amd_comgr_action_data_get_data(
@@ -314,8 +312,8 @@ int main(int Argc, char *Argv[]) {
314312
Status = amd_comgr_release_data(DataElement);
315313
checkError(Status, "amd_comgr_release_data");
316314

317-
if (BytesSize != 8) {
318-
printf("Arvhive host element size: %ld (expected 8)\n", BytesSize);
315+
if (!BytesSize) {
316+
printf("Arvhive host empty (expected non-empty)\n");
319317
exit(1);
320318
}
321319

0 commit comments

Comments
 (0)