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

fix for abseil not support avx compile #152

Merged
merged 1 commit into from
Oct 25, 2019
Merged

fix for abseil not support avx compile #152

merged 1 commit into from
Oct 25, 2019

Conversation

zh794390558
Copy link
Collaborator

Thank you for submitting a pull request! Please provide the following information for code review:

Pull Request Summary

fix this error for deltann-cpu-py3 image.

In file included from /delta/deltann//../tools/abseil-cpp/absl/strings/str_cat.h:63:0,
                 from /delta/deltann//../tools/abseil-cpp/absl/strings/internal/str_join_internal.h:43,
                 from /delta/deltann//../tools/abseil-cpp/absl/strings/str_join.h:59,
                 from /delta/deltann//../tools/tensorflow/tensorflow/core/lib/core/errors.h:21,
                 from /delta/deltann//../tools/tensorflow/tensorflow/core/framework/tensor_shape.h:23,
                 from /delta/deltann/core/shape.h:25,
                 from /delta/deltann/core/io.h:22,
                 from /delta/deltann/core/data.h:25,
                 from /delta/deltann/core/base_model.h:24,
                 from /delta/deltann/core/tfmodel.h:25,
                 from core/tfmodel.cc:23:
/delta/deltann//../tools/abseil-cpp/absl/strings/numbers.h: In function ‘size_t absl::numbers_internal::FastHexToBufferZeroPad16(uint64_t, char*)’:
/delta/deltann//../tools/abseil-cpp/absl/strings/numbers.h:197:12: error: ‘_mm_loadu_si64’ was not declared in this scope
   auto v = _mm_loadu_si64(reinterpret_cast<__m128i*>(&be));  // load lo dword
            ^~~~~~~~~~~~~~
/delta/deltann//../tools/abseil-cpp/absl/strings/numbers.h:197:12: note: suggested alternative: ‘_mm_loadl_epi64’
   auto v = _mm_loadu_si64(reinterpret_cast<__m128i*>(&be));  // load lo dword
            ^~~~~~~~~~~~~~
            _mm_loadl_epi64
Makefile:125: recipe for target '/delta/deltann/.gen/obj/core/tfmodel.o' failed
make: *** [/delta/deltann/.gen/obj/core/tfmodel.o] Error 1
build deltann error

Test Plan

dpl compile deltann pass.

@zh794390558 zh794390558 added Bug Something isn't working Deltann Dpl labels Oct 25, 2019
@zh794390558 zh794390558 added this to the v0.3.1 milestone Oct 25, 2019
@zh794390558 zh794390558 requested a review from GaryGao99 October 25, 2019 01:54
@zh794390558 zh794390558 self-assigned this Oct 25, 2019
@zh794390558 zh794390558 merged commit 628ba52 into master Oct 25, 2019
@zh794390558 zh794390558 deleted the fix branch October 25, 2019 03:04
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
Bug Something isn't working Deltann Dpl
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants