Skip to content

Remove LLVM 20 limitation#485

Open
MathewBensonCode wants to merge 1 commit intoJeanPhilippeKernel:developfrom
MathewBensonCode:update-llvm
Open

Remove LLVM 20 limitation#485
MathewBensonCode wants to merge 1 commit intoJeanPhilippeKernel:developfrom
MathewBensonCode:update-llvm

Conversation

@MathewBensonCode
Copy link
Contributor

  • clang-format ignore on CoroutineScheduler.h and VulkanDevice.h for operator bool()
  • update ubuntu CI workflow to use version 21.

Closes #484

if ($IsLinux) {
'/usr/bin/clang-format'
'/usr/bin/clang-format-21'
'/usr/bin/clang-format-20'

Choose a reason for hiding this comment

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

That means we want to remove the clang-format-20 from candidate list

Copy link
Contributor Author

Choose a reason for hiding this comment

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

I thought to leave it in as a precaution incase someone is still on the older platform. The main idea was to remove the restriction on newer versions but that compatibility with the older version is retained as specified in repoConfig.json

- clang-format ignore on CoroutineScheduler.h and VulkanDevice.h for
  operator bool()
- update ubuntu CI workflow to use version 21 as it tends to be behind
  in its inbuild version
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

area-linux Work on Linux system DevOps

Projects

Status: In Progress

Development

Successfully merging this pull request may close these issues.

Remove LLVM 20 limitation, LLVM 21 current default on many platforms

3 participants