On 12-01-06 10:38 AM, Mathieu Desnoyers wrote:
* Alexandre Montplaisir ([email protected]) wrote:
I tested it on x86_64, this should be tested on other arches before
merging, namely to make sure it catches GCC version below 4.4 on ARM.
We don't use these builtins on x86, we use our own implementation
instead.
This check does not take into account that some gcc versions
support the builtin, and compile something out of it, but the semantic
of the barrier is wrong, which will lead to runtime races.
Oh ok, I didn't know that. A straightforward GCC version-check would
probably be better in this case.
--
Alexandre Montplaisir
DORSAL lab,
École Polytechnique de Montréal
_______________________________________________
lttng-dev mailing list
[email protected]
http://lists.lttng.org/cgi-bin/mailman/listinfo/lttng-dev
