On 06/03/2014 11:21 AM, Kai Tietz wrote: > case AX_REG: > case DX_REG: > - return true; > + return (regno != DX_REG || !TARGET_64BIT || ix86_abi != MS_ABI);
You might as well eliminate the first test, and split the case entries: case AX_REG: return true; case DX_REG: return !TARGET_64BIT || ix86_abi != MS_ABI; Otherwise it looks ok. r~