"Bizarre implementation" of intrinsic Microsoft Visual C++ functions

Quote from CORE-3470:

Could someone check "bizarre implementation which follows Visual C++" of _bittestandcomplement, _bittestandreset, and _bittestandset in intrin_x86.h?
There should be:

_asm_("blah" : [retval] "=q" (retval), [a] "+mr" (*a) : [b] "r" (b));

instead of:

_asm_("blah" : [retval] "=q" (retval) : [a] "mr" (*a), [b] "r" (b));

i think?