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

compile with gcc-11 #93

Open
gled-rs opened this issue Jun 25, 2021 · 4 comments
Open

compile with gcc-11 #93

gled-rs opened this issue Jun 25, 2021 · 4 comments
Labels
documentation Improvements or additions to documentation enhancement New feature or request

Comments

@gled-rs
Copy link

gled-rs commented Jun 25, 2021

More of a notes than an issue but at this point tensorflow does not compile with gcc 11, so if needed here are the notes on how to compile.

Changes needed:

mkdir build
cd build
cmake ../

then in:

  • abseil-cpp/absl/synchronization/internal/graphcycles.h and ruy/ruy/block_map.h add at the top:
 #include <limits>
 #include <stdexcept>
  • xnnpack/src/xnnpack/intrinsics-polyfill.h replace line 15 :
    #if (defined(__GNUC__) && !defined(__clang__) && !defined(__INTEL_COMPILER)) || \
    by
    #if (defined(__GNUC__) && !defined(__clang__) && !defined(__INTEL_COMPILER) && __GNUC__ < 11) || \
@phlash phlash added documentation Improvements or additions to documentation enhancement New feature or request labels Jun 26, 2021
@BenBE
Copy link
Collaborator

BenBE commented Jun 26, 2021

Care to upstream those comments to the TensorFlow guys? Just in case there aren't open issues for those two changes there yet.

@gled-rs
Copy link
Author

gled-rs commented Jun 28, 2021

Care to upstream those comments to the TensorFlow guys? Just in case there aren't open issues for those two changes there yet.

they are well aware of the compile issue, that is where I found the workaround ( split in 2 different sources ).

I just thought of adding the note here to save some time for someone who would like to compile with gcc11 ( arch users for example ).

@phlash
Copy link
Collaborator

phlash commented Aug 24, 2022

Is this still an issue for newer Tensorflow versions? We're on 2.8.0 at present.

@BenBE
Copy link
Collaborator

BenBE commented Aug 24, 2022

AFAIR we now got problems with GCC 12, cf. #145

Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
documentation Improvements or additions to documentation enhancement New feature or request
Projects
None yet
Development

No branches or pull requests

3 participants