Skip to content

remove _GLIBCXX_USE_CXX11_ABI macro for libraries linking against the C library#2527

Merged
wanghan-iapcm merged 3 commits intodeepmodeling:develfrom
njzjz:remove_definitions
May 15, 2023
Merged

remove _GLIBCXX_USE_CXX11_ABI macro for libraries linking against the C library#2527
wanghan-iapcm merged 3 commits intodeepmodeling:develfrom
njzjz:remove_definitions

Conversation

@njzjz
Copy link
Member

@njzjz njzjz commented May 14, 2023

The libraries linking against the C library do not need to use the same _GLIBCXX_USE_CXX11_ABI as the TF.

… C library

The libraries linking against the C library do not need to use the same `_GLIBCXX_USE_CXX11_ABI` as the TF.

Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
@codecov
Copy link

codecov bot commented May 14, 2023

Codecov Report

Patch coverage has no change and project coverage change: -0.04 ⚠️

Comparison is base (a0f8767) 75.03% compared to head (6b02b21) 74.99%.

Additional details and impacted files
@@            Coverage Diff             @@
##            devel    #2527      +/-   ##
==========================================
- Coverage   75.03%   74.99%   -0.04%     
==========================================
  Files         228      228              
  Lines       23882    23882              
  Branches     1666     1680      +14     
==========================================
- Hits        17919    17910       -9     
- Misses       4889     4899      +10     
+ Partials     1074     1073       -1     

see 4 files with indirect coverage changes

☔ View full report in Codecov by Sentry.
📢 Do you have feedback about the report comment? Let us know in this issue.

njzjz added 2 commits May 14, 2023 21:01
Signed-off-by: Jinzhe Zeng <jinzhe.zeng@rutgers.edu>
build lmp in wheels with _GLIBCXX_USE_CXX11_ABI=0
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

[BUG] Fugaku(ARM node) Easy Install with pip lmp does not support deepmd pair style

2 participants