先看你代码里的几个明显问题：

• 语法错误：switch (instr): { 多了个冒号，正确写法是 switch (instr) {
• 提取立即数错误：比如10位立即数你用了instr & 15，这只取了低4位，应该取对应宽度的全部位
• 符号位判断和扩展的逻辑绕了弯路，其实用C语言的算术移位可以更简洁可靠地完成符号扩展

符号扩展的核心原理 #

对于n位有符号数，要扩展到16位，关键是把原数的符号位（第n-1位，从0开始计数）填充到16位中高于n-1的所有位。C语言中，有符号整数的右移操作是算术移位——会用符号位填充左侧空位，刚好能帮我们自动完成符号位的扩展。

修正后的完整代码 #

// 假设IMMI_10、IMMI_12、IMMI_15是定义好的常量
switch (instr_type) { // 变量名改成instr_type，避免和存储立即数的instr混淆
    case IMMI_10: {
        // 提取低10位→左移6位把符号位移到16位符号位→右移6位完成符号扩展
        signed short amount = (signed short)((instr & 0x3FF) << 6) >> 6;
        // 这里添加amount的业务逻辑
        break;
    }
    case IMMI_12: {
        // 16-12=4，左移4位再右移4位完成扩展
        signed short amount = (signed short)((instr & 0xFFF) << 4) >> 4;
        // 添加业务逻辑
        break;
    }
    case IMMI_15: {
        // 16-15=1，左移1位再右移1位完成扩展
        signed short amount = (signed short)((instr & 0x7FFF) << 1) >> 1;
        // 添加业务逻辑
        break;
    }
    default:
        // 处理未知指令类型的情况
        break;
}

代码细节解释 #

1. 提取对应位数的立即数：
   • 10位用掩码0x3FF（二进制10个连续的1），确保只保留instr的低10位
   • 12位用掩码0xFFF（12个连续的1）
   • 15位用掩码0x7FFF（15个连续的1）
2. 算术移位完成扩展：
   • 左移16 - width位：把原数的符号位移动到16位有符号整数（signed short）的符号位位置（第15位）
   • 右移16 - width位：因为是有符号数的右移，会自动用符号位填充左侧空位，一次性完成所有高位的符号扩展

通用封装写法（更易维护） #

如果需要处理任意宽度的有符号数扩展，可以封装成通用函数，减少重复代码：

signed short sign_extend(unsigned int instr, int width) {
    // 先提取指定宽度的数值
    unsigned int masked = instr & ((1U << width) - 1);
    // 通过算术移位完成16位符号扩展
    return (signed short)((masked << (16 - width)) >> (16 - width));
}

使用时直接调用即可：

signed short amount;
switch (instr_type) {
    case IMMI_10:
        amount = sign_extend(instr, 10);
        break;
    case IMMI_12:
        amount = sign_extend(instr, 12);
        break;
    case IMMI_15:
        amount = sign_extend(instr, 15);
        break;
    // 其他类型分支
}

内容的提问来源于stack exchange，提问作者Vineeth Sai