On 06/20/2011 06:33 PM, Richard Henderson wrote:
On 06/20/2011 09:22 AM, Andrew MacLeod wrote:
builtins.c:expand_builtin_lock_test_and_set (enum machine_mode mode, tree exp,
builtins.c: case BUILT_IN_LOCK_TEST_AND_SET_1:
builtins.c: case BUILT_IN_LOCK_TEST_AND_SET_2:
builtins.c: case BUILT_IN_LOCK_TEST_AND_SET_4:
whereas everything else is 'sync_lock_test_and_set'..
So i guess it falls to prior art... I assume Aldy just cut-and-pasted
for his new routine and just changed the names in the same format.
Heh. So this is really my fault. Ah well.
yeah, from 2005 :-) it seems to be localized to bultins.c,
sync-builtins.def, and c-family/c-common.c.
If you want to standardize it with SYNC_ for all cases, I will create
all the new ones that way.
I'm trying to avoid unnecessary noise on the branch. I'm bringing the
cxx-mem-model branch up to mainline revision now, so I can go and submit
a patch to fix the existing ones right now on mainline if you want...
turn them all into BUILT_IN_SYNC_LOCK_TEST_AND_SET or whatever they
need to be to match... then it will be right and wont affect me at all :-)
Andrew
