hoyFB added a comment. In D82822#2189692 <https://reviews.llvm.org/D82822#2189692>, @jdoerfert wrote:
> In D82822#2189366 <https://reviews.llvm.org/D82822#2189366>, @hoyFB wrote: > >> In D82822#2187576 <https://reviews.llvm.org/D82822#2187576>, @jdoerfert >> wrote: >> >>> @davezarzycki The bots reported this as well, didn't build MLIR locally, >>> this should have been fixed by 4d83aa4771d84940626d86c883193af390812281 >>> <https://reviews.llvm.org/rG4d83aa4771d84940626d86c883193af390812281> >> >> I was seeing the same issue with what @davezarzycki has encountered. Not >> sure why this is related to MLIR. The failure is cause by this line: >> >> 74: // CHECK-DEBUG-NEXT: [[OMP_GLOBAL_THREAD_NUM:%.*]] = call i32 >> @__kmpc_global_thread_num(%struct.ident_t* **@12**), !dbg !{{[0-9]*}} >> >> where `@12` is supposed to be defined as >> >> @11 = private unnamed_addr constant [90 x i8] >> c";llvm-project/clang/test/OpenMP/irbuilder_nested_parallel_for.c;parallel_for_1;85;1;;\00", >> align 1 >> @12 = private unnamed_addr global %struct.ident_t { i32 0, i32 2, i32 0, >> i32 0, i8* getelementptr inbounds ([90 x i8], [90 x i8]* @11, i32 0, i32 0) >> }, align 8 >> >> The exact match of `@12` seems a bit fragile to me since global data names >> could be changed from time to time. Can you please make it bit more robust? >> Thank > > I just created D85099 <https://reviews.llvm.org/D85099> to eliminate the > problem permanently. Feel free to take a look. Thanks for the quick turnaround! Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D82822/new/ https://reviews.llvm.org/D82822 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
