Skip to content

Remove LLVM 20 limitation #485

New issue

Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.

Sign up for GitHub

By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.

Already on GitHub? Sign in to your account

Open
MathewBensonCode wants to merge 1 commit into
base: develop
Choose a base branch
from
Open

Remove LLVM 20 limitation #485

Show file tree
Hide file tree
Changes from all commits
Commits
Show all changes
1 commit
Select commit
File filter

Filter by extension

Filter by extension
Conversations
Failed to load comments.
Loading
Jump to
Jump to file
Failed to load files.
Loading
Diff view
Diff view
11 changes: 6 additions & 5 deletions .github/workflows/job-cmakebuild-linux.yml
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -25,16 +25,17 @@ jobs:
- name: Register the Microsoft repository GPG keys
run: sudo dpkg -i packages-microsoft-prod.deb

- name: Install LLVM-20
- name: Install LLVM-21
run: |
wget https://apt.llvm.org/llvm.sh
chmod +x llvm.sh
sudo ./llvm.sh 20
sudo ./llvm.sh 21

- name: Update the list of packages after we added packages.microsoft.com
- name: Update the list of packages
run: sudo apt-get update
- name: Install development library
run: sudo apt-get install libasound2-dev libgl1-mesa-dev libpulse-dev libdbus-1-dev libx11-dev libxcursor-dev libxext-dev libxi-dev libxinerama-dev libxrandr-dev libxss-dev libxt-dev libxxf86vm-dev cmake apt-transport-https software-properties-common gcc g++ powershell clang-format-20 libwayland-dev libxkbcommon-dev dotnet-sdk-8.0

- name: Install Dependencies
run: sudo apt-get install libasound2-dev libgl1-mesa-dev libpulse-dev libdbus-1-dev libx11-dev libxcursor-dev libxext-dev libxi-dev libxinerama-dev libxrandr-dev libxss-dev libxt-dev libxxf86vm-dev cmake apt-transport-https software-properties-common gcc g++ powershell clang-format-21 glslang-tools libwayland-dev libxkbcommon-dev

- name: CMake Build
run: .\Scripts\BuildEngine.ps1 -Configurations ${{inputs.configuration}}
Expand Down
1 change: 1 addition & 0 deletions Scripts/Shared.ps1
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -201,6 +201,7 @@ function Find-ClangFormat () {
}
if ($IsLinux) {
'/usr/bin/clang-format'
'/usr/bin/clang-format-21'
'/usr/bin/clang-format-20'
Copy link
Owner

@JeanPhilippeKernel JeanPhilippeKernel Jan 16, 2026

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

@MathewBensonCode MathewBensonCode Jan 16, 2026

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'
Expand Down
3 changes: 2 additions & 1 deletion ZEngine/ZEngine/Core/CoroutineScheduler.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -14,11 +14,12 @@ namespace ZEngine::Core

ReadyCallback Ready = nullptr;
ExecuteCallback Action = nullptr;

// clang-format off
operator bool() noexcept
{
return (Ready && Action);
}
// clang-format on
};

struct CoroutineScheduler
Expand Down
6 changes: 4 additions & 2 deletions ZEngine/ZEngine/Hardwares/VulkanDevice.h
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -76,11 +76,12 @@ namespace ZEngine::Hardwares
BufferType Type = BufferType::UNKNOWN;
VkBuffer Handle = VK_NULL_HANDLE;
VmaAllocation Allocation = nullptr;

// clang-format off
operator bool() const
{
return (Handle != VK_NULL_HANDLE);
}
// clang-format on
};

struct BufferImage
Expand All @@ -90,11 +91,12 @@ namespace ZEngine::Hardwares
VkImageView ViewHandle{VK_NULL_HANDLE};
VkSampler Sampler{VK_NULL_HANDLE};
VmaAllocation Allocation{nullptr};

// clang-format off
operator bool() const
{
return (Handle != VK_NULL_HANDLE);
}
// clang-format on
};

struct IGraphicBuffer
Expand Down
2 changes: 1 addition & 1 deletion repoConfiguration.json
View file
Open in desktop
Original file line number Diff line number Diff line change
Expand Up @@ -24,7 +24,7 @@
},
"LLVM": {
"Version": "20.1.7",
"MaximumVersion": "20.2.0"
"MaximumVersion": "22.0.0"
}
}
}
Loading