agozillon added a comment. In D145815#4230780 <https://reviews.llvm.org/D145815#4230780>, @jeanPerier wrote:
> @agozillon, in the test added here (omp-frontend-forwarding.f90), I am seeing > failures in some patches windows pre-merge checks that I think are not > related to the patches. > Could you check if there is a stability/reproducibility issue with the > omp-frontend-forwarding.f90 on windows? > > The failure message look like: > > # command stderr: > > C:\ws\w8\llvm-project\premerge-checks\flang\test\Driver\omp-frontend-forwarding.f90:21:23: > error: CHECK-OPENMP-EMBED: expected string not found in input > ! CHECK-OPENMP-EMBED: "{{[^"]*}}clang-offload-packager" {{.*}} > "--image=file={{.*}}.bc,triple=amdgcn-amd-amdhsa,arch=gfx90a,kind=openmp" > ^ > <stdin>:6:435: note: scanning from here > "c:\\ws\\w8\\llvm-project\\premerge-checks\\build\\bin\\flang-new" "-fc1" > "-triple" "amdgcn-amd-amdhsa" "-emit-llvm-bc" "-fopenmp" "-mrelocation-model" > "pic" "-pic-level" "2" "-fopenmp-is-device" "-o" > "C:\\Users\\ContainerAdministrator\\AppData\\Local\\Temp\\lit-tmp-1koflt_t\\omp-frontend-forwarding-gfx90a-6808f0.bc" > "-x" "f95-cpp-input" > "C:\\ws\\w8\\llvm-project\\premerge-checks\\flang\\test\\Driver\\omp-frontend-forwarding.f90" > > > > > > ^ > <stdin>:7:266: note: possible intended match here > "c:\\program files\\llvm\\bin\\clang-offload-packager.exe" "-o" > "C:\\Users\\ContainerAdministrator\\AppData\\Local\\Temp\\lit-tmp-1koflt_t\\omp-frontend-forwarding-9beea6.out" > > "--image=file=C:\\Users\\ContainerAdministrator\\AppData\\Local\\Temp\\lit-tmp-1koflt_t\\omp-frontend-forwarding-gfx90a-6808f0.bc,triple=amdgcn-amd-amdhsa,arch=gfx90a,kind=openmp" > > I wonder if this is an issue with the ".exe" command suffix in the windows > output. > > Example of pre-merge failures: > > https://reviews.llvm.org/D146989 > https://buildkite.com/llvm-project/premerge-checks/builds/143821#01872b65-9b0c-457b-8714-c1f0ca00d02b > > or > https://reviews.llvm.org/D147130 > https://buildkite.com/llvm-project/premerge-checks/builds/143873#01872ccb-9251-499a-b9fd-9155e3ffb1f1 Hi @jeanPerier, Thank you! I noticed this today as well and I'm currently looking into it, i have a fix upcoming, just forcing another patch (https://reviews.llvm.org/D144896) to test it via buildbot as I have no easy access to a windows machine to test it at the moment, and yes @jhuber6 is correct, it appears to be the .exe postfix on windows! I'll submit the fix as soon as it clears the buildbot. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D145815/new/ https://reviews.llvm.org/D145815 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits