[PATCH] D85938: [OpenMP][NFC] Update check lines after D85099
Johannes Doerfert via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Aug 13 18:33:59 PDT 2020
jdoerfert added a comment.
Root cause:
We sometimes, depending on the invocation and potentially other factors, use a full path and sometimes not:
So wen I run this with clang locally I get this:
@98 = private unnamed_addr constant [98 x i8] c";/data/src/llvm-project/clang/test/OpenMP/irbuilder_nested_parallel_for.c;parallel_for_2;294;16;;\00", align 1
@100 = private unnamed_addr constant %struct.ident_t { i32 0, i32 2, i32 0, i32 0, i8* getelementptr inbounds ([98 x i8], [98 x i8]* @98, i32 0, i32 0) }, align 8
@101 = private unnamed_addr constant [92 x i8] c";src/llvm-project/clang/test/OpenMP/irbuilder_nested_parallel_for.c;parallel_for_2;294;16;;\00", align 1
@103 = private unnamed_addr constant %struct.ident_t { i32 0, i32 2, i32 0, i32 0, i8* getelementptr inbounds ([92 x i8], [92 x i8]* @101, i32 0, i32 0) }, align 8
Note the 6 missing characters `/data/` in the second string. Because they are missing, we generate two globals for this. When I run the same test with the update script I get one global as they agree on the location.
Looking for the location that strips the path now.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D85938/new/
https://reviews.llvm.org/D85938
More information about the cfe-commits
mailing list