Skip to content

Conversation

omjavaid
Copy link
Contributor

@omjavaid omjavaid commented Sep 4, 2025

This is a follow up to #586.

Fixes 60d2141.

We forgot to fix the lnt path to work on both windows and linux.

On Windows, it's Scripts/lnt, on Linux it's bin/lnt.

This is a follow up to llvm#586.

Fixes 60d2141.

We forgot to fix the lnt path to work on both windows and linux.

On Windows, it's Scripts/lnt, on Linux it's bin/lnt.
Copy link
Contributor

@DavidSpickett DavidSpickett left a comment

Choose a reason for hiding this comment

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

LGTM

@omjavaid omjavaid merged commit 02f18da into llvm:main Sep 4, 2025
3 checks passed
vvereschaka pushed a commit to vvereschaka/llvm-zorg that referenced this pull request Sep 25, 2025
This is a follow up to llvm#586.

Fixes 60d2141. (llvm#576)

We forgot to fix the lnt path to work on both windows and linux.

On Windows, it's Scripts/lnt, on Linux it's bin/lnt.
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants