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

Legacy removing #218

Closed
ermig1979 opened this issue Jun 29, 2022 · 4 comments
Closed

Legacy removing #218

ermig1979 opened this issue Jun 29, 2022 · 4 comments

Comments

@ermig1979
Copy link
Owner

ermig1979 commented Jun 29, 2022

Simd is being developing about 12 years.
And some of its components are not actual now.
I think that I can remove some of functionality that are not updated for many years.
This list is not full and may be updated:

  1. Support of PPC architecture.
  2. Data tests (they were developed for PPC porting).
  3. AVX-512F optimizations (merge them to AVX-512BW optimizations).
  4. SSE2 optimizations (merge them to SSE4.1 optimizations).
  5. AVX optimizations (merge them to AVX2 optimizations).
  6. EdgeBackground functions.
  7. SvmSumLinear function.
  8. Interference functions.
  9. SimdTensorFormatType - remove all types besides NHWC and NCHW.
  10. SimdSynetReorderImage and SimdSynetReorderFilter functions.
  11. SimdSynetSpecifyTensorFormat and SimdSynetTensorAlignment functions.
  12. SynetAddBias - remove support of all formats besides NHWC and NCHW.
  13. SynetScaleLayerForward - remove support of all formats besides NHWC and NCHW.
  14. SynetPreluLayerForward - remove support of all formats besides NHWC and NCHW.
  15. SynetFusedLayerForward(0-9) - remove support of all formats besides NHWC and NCHW.
  16. SimdYuv444pToBgr, SimdYuv422pToBgr, SimdYuv420pToBgr.
  17. SimdYuv444pToBgra, SimdYuv422pToBgra, SimdYuv420pToBgra
@mikeversteeg
Copy link

I have no problems with this. Another way of cleaning up is to remove YUV420P functions if there is a similar YUV422P version.

@ermig1979
Copy link
Owner Author

I want to note that YUV420P is not the same as YUV422P.

@mikeversteeg
Copy link

Can't you convert 422 to 420 by doubling the U & V stride?

@mikeversteeg
Copy link

Oh no, that's to make a 420 from a 422. What we need here is to tell stride to increment at half speed. The rest of the code can remain the same. I'd definitely want 422 supported.

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

No branches or pull requests

2 participants