From: FFADO <ffa...@ff...> - 2010-05-09 09:33:57
|
#280: atomic.h should use GCC builtins ----------------------+----------------------------------------------------- Reporter: ppalmers | Owner: Type: bug | Status: new Priority: minor | Milestone: FFADO 2.x Component: generic | Version: FFADO 2.0.0 Keywords: | Device_name: ----------------------+----------------------------------------------------- Comment from Lennart Poettering: 5) Atomic.h should be updated to make use of the gcc __sync builtins when available. They are nowadays supported on almost all archs and support the full set of atomic ops. If you have them then you don't need implement those operations manually based on the CAS. I have seen that you currently use __sync_bool_compare_and_swap() as a fallback. But it should be the first choice nowadays. Both PA and glib have been using __sync now for a while and by default. While we initially had problems with some of the code generated this has long been fixed since then in gcc. -- Ticket URL: <http://subversion.ffado.org/ticket/280> FFADO <http://subversion.ffado.org/index.fcgi> Free Firewire Audio Drivers |