Skip to content

Conversation

Megamouse
Copy link
Contributor

@Megamouse Megamouse commented Mar 4, 2025

  • Updates LLVM to 21.1.3

TODO:

  • update docker
  • upload precompiled files

@Megamouse Megamouse added the Build and CI Anything related to the build process and continuous integration label Mar 4, 2025
@Megamouse Megamouse force-pushed the llvm20 branch 5 times, most recently from 1156c4b to 702fab7 Compare March 10, 2025 20:14
@Megamouse Megamouse changed the title Update llvm to 20.1.0 Update llvm to 20.1.1 Mar 20, 2025
@cipherxof
Copy link
Contributor

I'm getting slightly worse performance in LLVM 20 compared to 19, which already had worse AVX512 performance than LLVM 17 (#16733). Tested on a 8945HS.

LLVM 19:

image

LLVM 20:

image

@Megamouse Megamouse changed the title Update llvm to 20.1.1 Update llvm to 20.1.2 Apr 9, 2025
@crashGG
Copy link

crashGG commented Apr 9, 2025

But there is no
https://github.com/RPCS3/llvm-mirror/releases/download/custom-build-win-20.1.2/llvmlibs_mt.7z
So can't build the win64 version to test.
Can anyone help?

@Megamouse
Copy link
Contributor Author

There won't be a prebuilt until we decide to merge this.

@Megamouse Megamouse changed the title Update llvm to 20.1.2 Update llvm to 20.1.3 Apr 24, 2025
@Megamouse Megamouse changed the title Update llvm to 20.1.3 Update llvm to 20.1.4 May 14, 2025
@Megamouse Megamouse changed the title Update llvm to 20.1.4 Update llvm to 20.1.5 May 19, 2025
@Megamouse Megamouse changed the title Update llvm to 20.1.5 Update llvm to 20.1.6 May 28, 2025
@Megamouse Megamouse force-pushed the llvm20 branch 2 times, most recently from 2ace833 to e46d46d Compare June 1, 2025 16:14
@Megamouse Megamouse changed the title Update llvm to 20.1.6 Update llvm to 20.1.7 Jul 3, 2025
@Darkhost1999
Copy link
Contributor

Rpcs3_default.props doesn't change from llvm 20.1.7 to 20.1.8.
Are minor performance regressions a stopper for PRs like these?
I thought the goal is to test stability. Performance is allowed to fluctuate a little bit, especially with numbers as high as 120+

@kd-11
Copy link
Contributor

kd-11 commented Jul 10, 2025

Rpcs3_default.props doesn't change from llvm 20.1.7 to 20.1.8. Are minor performance regressions a stopper for PRs like these? I thought the goal is to test stability. Performance is allowed to fluctuate a little bit, especially with numbers as high as 120+

It's incremental just like optimizations are. We already had another drop coming from 17 to 19 then 19 to 20. Obviously something is wrong and needs to be identified. Otherwise by the time we reach llvm30 we'll have lost a lot of performance without making any significant code changes in our own code.

When no code changes product side have been made, regressions are a lot more concerning than when our own code is being modified because then we know the pros vs cons. We never commit our own code changes if they do nothing but hurt performance - the amount of experimental code that gets thrown out could probably create another (horrible) PS3 emulator.

In this case we have a change that provides zero pros, just cons, making it not worth the effort. If the cause is identified, we can proceed just to be up to date with the latest version of llvm. Something I am curious about is whether the problem goes away if we use the clang64 windows binaries?

@Darkhost1999
Copy link
Contributor

Something I am curious about is whether the problem goes away if we use the clang64 windows binaries?

Wish I could try. But I tend to observe the same "problem" as previously reported with the mgs4 test booting any game on the exact same binary of rpcs3. Let alone a different binary.
I'm not any good at performance tests at all.

@Megamouse Megamouse force-pushed the llvm20 branch 2 times, most recently from d6a62e8 to 2516cac Compare July 11, 2025 16:49
@Megamouse Megamouse changed the title Update llvm to 20.1.7 Update llvm to 20.1.8 Jul 11, 2025
@JohnHDev
Copy link

Would be good to see if this update fixes this issue: #16676

Happy to try an ARM Mac build if you wanted someone to try it.

@Megamouse Megamouse force-pushed the llvm20 branch 2 times, most recently from 5c2b2d2 to 32f1e4c Compare September 24, 2025 06:15
@Megamouse Megamouse changed the title Update llvm to 20.1.8 Update llvm to 21.1.2 Sep 24, 2025
@Megamouse Megamouse force-pushed the llvm20 branch 4 times, most recently from 07ee7f9 to bb468b8 Compare September 24, 2025 06:57
@Megamouse Megamouse changed the title Update llvm to 21.1.2 Update llvm to 21.1.3 Oct 10, 2025
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

Build and CI Anything related to the build process and continuous integration

Projects

None yet

Development

Successfully merging this pull request may close these issues.

6 participants