search

ImageOptim / libimagequant

Palette quantization library that powers pngquant and other PNG optimizers
https://pngquant.org/lib
Other
780 stars 132 forks source link

Update build.rs #33

Closed XhmikosR closed 4 years ago

XhmikosR commented 4 years ago

x64 builds don't need /arch:SSE2 specified

Noticed it on https://github.com/kornelski/pngquant/issues/312#issuecomment-568888500

I only tested x64 builds, and I'm not familiar with rust syntax so let me know if this doesn't do the job.

Noticed it on pngquant msvc branch. Any reason why you don't use the master branch and merge the msvc changes there @kornelski ?

kornelski commented 4 years ago

The flag may not be needed, but USE_SSE is.

kornelski commented 4 years ago

Visual Studio has poor C support, and I don't want VS hacks in the codebase.

XhmikosR commented 4 years ago

But the /arch:SSSE2 is MSVC specific compiler option.