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

cmake: pass -DINTEL* to gf-complete cflags #10956

Merged
merged 1 commit into from Sep 2, 2016
Merged

cmake: pass -DINTEL* to gf-complete cflags #10956

merged 1 commit into from Sep 2, 2016

Conversation

tchaikov
Copy link
Contributor

@tchaikov tchaikov commented Sep 2, 2016

  • in addition to the -march=foobar, we also need to pass -DFOOBAR to
    gf-complete sources' cflags, so that the SIMD instructions can be
    generated as expected.
  • also extract the compiler support for instruction extensions detections
    into SIMDExt.cmake.
  • and only check the compiler support if CMAKE_SYSTEM_PROCESSOR matches
    with the instruction-set/arch to be checked.

Signed-off-by: tone.zhang tone.zhang@linaro.org
Signed-off-by: Kefu Chai kchai@redhat.com

@ghost
Copy link

ghost commented Sep 2, 2016

Reviewed-by: Loic Dachary <ldachary@redhat.com>

* in addition to the -march=foobar, we also need to pass -DFOOBAR to
  gf-complete sources' cflags, so that the SIMD instructions can be
  generated as expected.
* also extract the compiler support for instruction extensions detections
  into SIMDExt.cmake.
* and only check the compiler support if CMAKE_SYSTEM_PROCESSOR matches
  with the instruction-set/arch to be checked.

Signed-off-by: tone.zhang <tone.zhang@linaro.org>
Signed-off-by: Kefu Chai <kchai@redhat.com>
@tchaikov
Copy link
Contributor Author

tchaikov commented Sep 2, 2016

@dachary thanks for your review. i just realized that we should include(SIMDExt) in erasure/CMakeLists.txt instead of in erasure/jerasure/CMakeLists.txt, so the list the plugin flavours can be initialized correctly.

@tchaikov tchaikov merged commit 5e804ba into master Sep 2, 2016
@tchaikov tchaikov deleted the wip-cmake branch September 2, 2016 09:22
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Projects
None yet
1 participant