Skip to content

Conversation

aganea
Copy link
Member

@aganea aganea commented Jul 18, 2025

The checks for detecting if clang-cl and lld-link are in %PATH were wrong.

This fixes the comment in #135446 (comment)

Copy link
Collaborator

@zmodem zmodem left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

lgtm

(Sorry for the late review; I was basically offline for the past four weeks.)

@github-project-automation github-project-automation bot moved this from Needs Triage to Needs Merge in LLVM Release Status Aug 18, 2025
@aganea aganea enabled auto-merge (squash) August 18, 2025 14:02
@tru tru moved this from Needs Merge to Needs Backport PR in LLVM Release Status Aug 19, 2025
@zmodem
Copy link
Collaborator

zmodem commented Aug 19, 2025

Doesn't look like it's auto-merging. Maybe hit the button manually?

@tru
Copy link
Collaborator

tru commented Aug 19, 2025

It didn't merge because a test failed.

@aganea aganea merged commit 13391ce into llvm:main Aug 19, 2025
9 checks passed
@github-project-automation github-project-automation bot moved this from Needs Backport PR to Done in LLVM Release Status Aug 19, 2025
@llvm-ci
Copy link
Collaborator

llvm-ci commented Aug 19, 2025

LLVM Buildbot has detected a new failure on builder mlir-nvidia-gcc7 running on mlir-nvidia while building llvm at step 7 "test-build-check-mlir-build-only-check-mlir".

Full details are available at: https://lab.llvm.org/buildbot/#/builders/116/builds/17152

Here is the relevant piece of the build log for the reference
Step 7 (test-build-check-mlir-build-only-check-mlir) failure: test (failure)
******************** TEST 'MLIR :: Integration/GPU/Vulkan/vector-deinterleave.mlir' FAILED ********************
Exit Code: 1

Command Output (stdout):
--
# RUN: at line 1
/vol/worker/mlir-nvidia/mlir-nvidia-gcc7/llvm.obj/bin/mlir-opt /vol/worker/mlir-nvidia/mlir-nvidia-gcc7/llvm.src/mlir/test/Integration/GPU/Vulkan/vector-deinterleave.mlir -test-vulkan-runner-pipeline    | /vol/worker/mlir-nvidia/mlir-nvidia-gcc7/llvm.obj/bin/mlir-runner -      --shared-libs=/vol/worker/mlir-nvidia/mlir-nvidia-gcc7/llvm.obj/lib/libmlir_vulkan_runtime.so,/vol/worker/mlir-nvidia/mlir-nvidia-gcc7/llvm.obj/lib/libmlir_runner_utils.so      --entry-point-result=void | /vol/worker/mlir-nvidia/mlir-nvidia-gcc7/llvm.obj/bin/FileCheck /vol/worker/mlir-nvidia/mlir-nvidia-gcc7/llvm.src/mlir/test/Integration/GPU/Vulkan/vector-deinterleave.mlir
# executed command: /vol/worker/mlir-nvidia/mlir-nvidia-gcc7/llvm.obj/bin/mlir-opt /vol/worker/mlir-nvidia/mlir-nvidia-gcc7/llvm.src/mlir/test/Integration/GPU/Vulkan/vector-deinterleave.mlir -test-vulkan-runner-pipeline
# executed command: /vol/worker/mlir-nvidia/mlir-nvidia-gcc7/llvm.obj/bin/mlir-runner - --shared-libs=/vol/worker/mlir-nvidia/mlir-nvidia-gcc7/llvm.obj/lib/libmlir_vulkan_runtime.so,/vol/worker/mlir-nvidia/mlir-nvidia-gcc7/llvm.obj/lib/libmlir_runner_utils.so --entry-point-result=void
# .---command stderr------------
# |  failed with error code 0 when executing vkCreateDevicerunOnVulkan failed
# `-----------------------------
# executed command: /vol/worker/mlir-nvidia/mlir-nvidia-gcc7/llvm.obj/bin/FileCheck /vol/worker/mlir-nvidia/mlir-nvidia-gcc7/llvm.src/mlir/test/Integration/GPU/Vulkan/vector-deinterleave.mlir
# .---command stderr------------
# | /vol/worker/mlir-nvidia/mlir-nvidia-gcc7/llvm.src/mlir/test/Integration/GPU/Vulkan/vector-deinterleave.mlir:6:11: error: CHECK: expected string not found in input
# | // CHECK: [0, 2]
# |           ^
# | <stdin>:1:1: note: scanning from here
# | Unranked Memref base@ = 0x5cbfed9f0820 rank = 1 offset = 0 sizes = [2] strides = [1] data = 
# | ^
# | <stdin>:2:1: note: possible intended match here
# | [0, 0]
# | ^
# | 
# | Input file: <stdin>
# | Check file: /vol/worker/mlir-nvidia/mlir-nvidia-gcc7/llvm.src/mlir/test/Integration/GPU/Vulkan/vector-deinterleave.mlir
# | 
# | -dump-input=help explains the following input dump.
# | 
# | Input was:
# | <<<<<<
# |            1: Unranked Memref base@ = 0x5cbfed9f0820 rank = 1 offset = 0 sizes = [2] strides = [1] data =  
# | check:6'0     X~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~ error: no match found
# |            2: [0, 0] 
# | check:6'0     ~~~~~~~
# | check:6'1     ?       possible intended match
# |            3: Unranked Memref base@ = 0x5cbfeda30a20 rank = 1 offset = 0 sizes = [2] strides = [1] data =  
# | check:6'0     ~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~~
# |            4: [0, 0] 
# | check:6'0     ~~~~~~~
# | >>>>>>
# `-----------------------------
# error: command failed with exit status: 1

--

********************


@aganea aganea deleted the fix/windows_release_script_msvc_fix branch August 19, 2025 14:09
@aganea
Copy link
Member Author

aganea commented Aug 19, 2025

/cherry-pick 13391ce

@llvmbot
Copy link
Member

llvmbot commented Aug 19, 2025

/pull-request #154399

tru pushed a commit to llvmbot/llvm-project that referenced this pull request Aug 21, 2025
… in PATH (llvm#149597)

The checks for detecting if `clang-cl` and `lld-link` are in `%PATH`
were wrong.

This fixes the comment in
llvm#135446 (comment)

(cherry picked from commit 13391ce)
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
Development

Successfully merging this pull request may close these issues.

5 participants