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.

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

Jump to bottom

[UWP] Switch the compiler to Clang for better performance #686

Open
mattleibow opened this issue Nov 15, 2018 · 8 comments
Open

[UWP] Switch the compiler to Clang for better performance #686

mattleibow opened this issue Nov 15, 2018 · 8 comments
Assignees
@mattleibow
Labels
area/Build area/libSkiaSharp.native os/Windows-Universal-UWP type/feature-request

Comments

@mattleibow
Copy link
Contributor

mattleibow commented Nov 15, 2018 •
edited by xamarin-release-manager
Loading

Description

Google has indicated that they only optimize for the clang compiler:

VS bug #730674
@mattleibow mattleibow changed the title Switch the compiler to Clang Switch the compiler to Clang for better performance Nov 15, 2018
@mattleibow mattleibow mentioned this issue Jan 10, 2019
Closed
5 tasks
mattleibow added a commit that referenced this issue Jan 24, 2019
@mattleibow
Switch to clang 3.8 on linux. #686
99338be
This was referenced Jan 24, 2019
Merged
Closed
mattleibow added a commit that referenced this issue Feb 5, 2019
@mattleibow
Added Azure DevOps Pipeline (#776)
d6f333a 
* Trying to get an older linux build
* Switch to clang 3.8 on linux. #686
* Statically link things for linux
* Set up CI with Azure Pipelines
* Updated the tizen install script to work on macos and linux
* Make sure to install the version of the Android NDK we support
* Added missing [SkippableFact] attribute
@dtoppani-twist
Copy link

I see work to use clang on linux to get more performance - is it possible to do so on Windows or there are specific blockers?

This was referenced Nov 11, 2019
Merged
Merged
@mattleibow
Copy link
Contributor Author

It has been a long time, but we are moving Win32 to Clang. Noticing vast improvements already, just check out #758. Need a little more time for UWP, but that can come later as there are probably fewer users of that platform.

As soon as #1007 is merged, I'll get a preview release out for the world.

@daltonks
Copy link
Contributor

Hey Matt,
I am a user of UWP and would really appreciate Clang there too. I might be one of the few though 😄

@mfkl
Copy link

mfkl commented Jul 7, 2020

Hi Matthew,
This landed recently, thought you might find it useful (it uses llvm-mingw).

@mattleibow
Copy link
Contributor Author

Thanks! I must really get to this sometime.

@daltonks
Copy link
Contributor

I'm working on a feature for my UWP app that could really use this (rotated images). It's unfortunate how slow the rendering is at the moment.

@mattleibow mattleibow changed the title Switch the compiler to Clang for better performance [UWP] Switch the compiler to Clang for better performance Jan 26, 2021
@rubiomiguel06
Copy link

I am also one of the few... But I am really having a hard time with the rendering performance of rotated images with Skia. Is this planned to be solved any time soon?
Thanks @mattleibow for all your research and work on this topic

@daltonks
Copy link
Contributor

I'm also still struggling with this on UWP. It's unfortunate that my desktop is slower than my Android phone, and users notice it too.
I'll eventually move to MAUI (hoping that it's better), but that's a long way off.

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Assignees

@mattleibow mattleibow

Labels
area/Build area/libSkiaSharp.native os/Windows-Universal-UWP type/feature-request
Projects
SkiaSharp Backlog
Status: New
Milestone
No milestone
Development

No branches or pull requests
7 participants
@Redth @xamarin-release-manager @daltonks @mattleibow @mfkl @dtoppani-twist @rubiomiguel06