Skip to content
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.

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

native cmake fixes + add native llvm 14 #6198

Merged
merged 7 commits into from
Sep 15, 2024

Conversation

th0ma7
Copy link
Contributor

@th0ma7 th0ma7 commented Aug 20, 2024

Description

  1. Noticed issues with native cmake building
  2. Adding native llvm 14 - This is to be used as pre-compiled released on spksrc for fastening the IGC build from [WIP] Enable opencl on ffmpeg5-6 with DSM >= 7 #6166

Checklist

  • Build rule all-supported completed successfully
  • New installation of package completed successfully
  • Package upgrade completed successfully (Manually install the package again)
  • Package functionality was tested
  • Any needed documentation is updated/created

Type of change

  • Bug fix
  • New Package
  • Package update
  • Includes small framework changes
  • This change requires a documentation update (e.g. Wiki)

@th0ma7 th0ma7 requested a review from hgy59 August 20, 2024 13:12
@th0ma7 th0ma7 self-assigned this Aug 20, 2024
@th0ma7
Copy link
Contributor Author

th0ma7 commented Aug 20, 2024

@hgy59 would you have cycles to test building the native llvm-14 and generate a resulting package similar to llvm-9.0 along with related native/llvm-14.0.

I'm finally inches away from enabling opencl on ffmpeg x64 builds, but similar to llvm-9.0 I require NATIVE clang|llvm-tlbgen binaries matching the built version, thus this addition into native.

Thnx in advance!

@th0ma7
Copy link
Contributor Author

th0ma7 commented Sep 3, 2024

@hgy59 friendly reminder when you have a moment to build a llvm-14 package to be re-used, thnx.

@hgy59
Copy link
Contributor

hgy59 commented Sep 4, 2024

@hgy59 friendly reminder when you have a moment to build a llvm-14 package to be re-used, thnx.

@th0ma7 you find the build as https://github.com/SynoCommunity/spksrc/releases/download/native%2Fllvm/native-llvm-14.0.5.tar.gz

Unfortunately there is no build log file created for native packages. So I have not a clue about build warnings...

@th0ma7
Copy link
Contributor Author

th0ma7 commented Sep 4, 2024

@hgy59 friendly reminder when you have a moment to build a llvm-14 package to be re-used, thnx.

@th0ma7 you find the build as https://github.com/SynoCommunity/spksrc/releases/download/native%2Fllvm/native-llvm-14.0.5.tar.gz

Unfortunately there is no build log file created for native packages. So I have not a clue about build warnings...

Indeed that's something we should add. Interestingly, when launched from an spk build logs does get generated btw...

@th0ma7
Copy link
Contributor Author

th0ma7 commented Sep 14, 2024

@hgy59 I just captured that the file is now available (thnx a lot!) although I can't find it in our release page?
https://github.com/SynoCommunity/spksrc/releases

@th0ma7
Copy link
Contributor Author

th0ma7 commented Sep 14, 2024

Don't bother, I just found it out at https://github.com/SynoCommunity/spksrc/releases/tag/native%2Fllvm
Again thnx.

@th0ma7
Copy link
Contributor Author

th0ma7 commented Sep 14, 2024

@hgy59 if you have a moment for a quick review, considering your commit addition I presume it to be all ok.

@th0ma7 th0ma7 merged commit 3bc0dc1 into SynoCommunity:master Sep 15, 2024
17 checks passed
@th0ma7 th0ma7 deleted the native-llvm-14 branch September 15, 2024 12:27
@hgy59 hgy59 mentioned this pull request Sep 16, 2024
6 tasks
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