Skip to content

Add <CGAL/atomic.h> and CGAL_CAN_USE_CXX11_ATOMIC#642

Merged
sloriot merged 5 commits intoCGAL:masterfrom
lrineau:Install-add_CGAL_CAN_USE_CXX11_ATOMIC-lrineau
Jan 21, 2016
Merged

Add <CGAL/atomic.h> and CGAL_CAN_USE_CXX11_ATOMIC#642
sloriot merged 5 commits intoCGAL:masterfrom
lrineau:Install-add_CGAL_CAN_USE_CXX11_ATOMIC-lrineau

Conversation

@lrineau
Copy link
Copy Markdown
Member

@lrineau lrineau commented Jan 15, 2016

  • Change the way the macros CGAL_CAN_USE_CXX11_THREAD_LOCAL and
    CGAL_CAN_USE_CXX11_MUTEX are defined, so that other compilers are
    also supported.
  • Add the macro CGAL_CAN_USE_CXX11_ATOMIC and the header
    <CGAL/atomic.h>.

- Change the way the macros `CGAL_CAN_USE_CXX11_THREAD_LOCAL` and
  `CGAL_CAN_USE_CXX11_MUTEX` are defined, so that other compilers are
  also supported.

- Add the macro `CGAL_CAN_USE_CXX11_ATOMIC` and the header
  `<CGAL/atomic.h>`.
@lrineau lrineau added this to the 4.8-beta milestone Jan 15, 2016
@lrineau
Copy link
Copy Markdown
Member Author

lrineau commented Jan 15, 2016

Merged in CGAL-4.8-Ic-101.

@sloriot
Copy link
Copy Markdown
Member

sloriot commented Jan 18, 2016

Unless I'm wrong, all the runtimes errors in 101 are due to this branch

@sloriot
Copy link
Copy Markdown
Member

sloriot commented Jan 18, 2016

My bad, problems come from the branch modifying CORE.

@sloriot
Copy link
Copy Markdown
Member

sloriot commented Jan 18, 2016

Merged in CGAL-4.8-Ic-103

The implementation is simplified.

If no implementation of `::atomic` is available, `CGAL_NO_ATOMIC` is
defined to an error message (replaced the `#error`).
@sloriot
Copy link
Copy Markdown
Member

sloriot commented Jan 19, 2016

Merged in CGAL-4.8-Ic-104

@sloriot sloriot self-assigned this Jan 21, 2016
sloriot added a commit that referenced this pull request Jan 21, 2016
…TOMIC-lrineau

Add <CGAL/atomic.h> and CGAL_CAN_USE_CXX11_ATOMIC
@sloriot sloriot merged commit a5ab328 into CGAL:master Jan 21, 2016
@sloriot sloriot deleted the Install-add_CGAL_CAN_USE_CXX11_ATOMIC-lrineau branch January 21, 2016 08:25
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.

2 participants