[GitHub] [arrow-rs] roee88 edited a comment on pull request #324: Enable wasm32 as a target architecture for the SIMD feature

[GitBox <gi...@apache.org>]

roee88 edited a comment on pull request #324:
URL: https://github.com/apache/arrow-rs/pull/324#issuecomment-846508588


   Sure. Though I don't think that the link you shared actually affects it. Originally there was no feature flag for simd and it was always enabled for x86 and x86_64. Then, a feature flag was added on top to allow disabling simd, it was then set to disabled by default, then cfg_aliases  was used (no functionality change), and finally aarch64 was added to the list of architectures. 
   
   So I might be wrong but it all seems to be for historic reasons. However, thus applies only as long as it's disabled by default. If at some point it will change to enable by default it would need to be enabled by default only for known supported architectures and disabled by default otherwise. Is that something that's possible to implement easily?


-- 
This is an automated message from the Apache Git Service.
To respond to the message, please log on to GitHub and use the
URL above to go to the specific comment.

For queries about this service, please contact Infrastructure at:
users@infra.apache.org