This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG5704630ec40d: [RISCV] Remove redundant _ta suffix in RVV intrinsics builtins. NFC (authored by eopXD).
Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D154693/new/ https://reviews.llvm.org/D154693 Files: clang/lib/Sema/SemaChecking.cpp clang/lib/Support/RISCVVIntrinsicUtils.cpp Index: clang/lib/Support/RISCVVIntrinsicUtils.cpp =================================================================== --- clang/lib/Support/RISCVVIntrinsicUtils.cpp +++ clang/lib/Support/RISCVVIntrinsicUtils.cpp @@ -1080,10 +1080,9 @@ } else { if (PolicyAttrs.isTUPolicy()) appendPolicySuffix("_tu"); - else if (PolicyAttrs.isTAPolicy()) { - if (HasPolicy) - BuiltinName += "_ta"; - } else + else if (PolicyAttrs.isTAPolicy()) // no suffix needed + return; + else llvm_unreachable("Unhandled policy condition"); } } Index: clang/lib/Sema/SemaChecking.cpp =================================================================== --- clang/lib/Sema/SemaChecking.cpp +++ clang/lib/Sema/SemaChecking.cpp @@ -4697,24 +4697,24 @@ case RISCV::BI__builtin_riscv_aes64ks1i_64: return SemaBuiltinConstantArgRange(TheCall, 1, 0, 10); // Check if value range for vxrm is in [0, 3] - case RISCVVector::BI__builtin_rvv_vaaddu_vv_ta: - case RISCVVector::BI__builtin_rvv_vaaddu_vx_ta: - case RISCVVector::BI__builtin_rvv_vaadd_vv_ta: - case RISCVVector::BI__builtin_rvv_vaadd_vx_ta: - case RISCVVector::BI__builtin_rvv_vasubu_vv_ta: - case RISCVVector::BI__builtin_rvv_vasubu_vx_ta: - case RISCVVector::BI__builtin_rvv_vasub_vv_ta: - case RISCVVector::BI__builtin_rvv_vasub_vx_ta: - case RISCVVector::BI__builtin_rvv_vsmul_vv_ta: - case RISCVVector::BI__builtin_rvv_vsmul_vx_ta: - case RISCVVector::BI__builtin_rvv_vssra_vv_ta: - case RISCVVector::BI__builtin_rvv_vssra_vx_ta: - case RISCVVector::BI__builtin_rvv_vssrl_vv_ta: - case RISCVVector::BI__builtin_rvv_vssrl_vx_ta: - case RISCVVector::BI__builtin_rvv_vnclip_wv_ta: - case RISCVVector::BI__builtin_rvv_vnclip_wx_ta: - case RISCVVector::BI__builtin_rvv_vnclipu_wv_ta: - case RISCVVector::BI__builtin_rvv_vnclipu_wx_ta: + case RISCVVector::BI__builtin_rvv_vaaddu_vv: + case RISCVVector::BI__builtin_rvv_vaaddu_vx: + case RISCVVector::BI__builtin_rvv_vaadd_vv: + case RISCVVector::BI__builtin_rvv_vaadd_vx: + case RISCVVector::BI__builtin_rvv_vasubu_vv: + case RISCVVector::BI__builtin_rvv_vasubu_vx: + case RISCVVector::BI__builtin_rvv_vasub_vv: + case RISCVVector::BI__builtin_rvv_vasub_vx: + case RISCVVector::BI__builtin_rvv_vsmul_vv: + case RISCVVector::BI__builtin_rvv_vsmul_vx: + case RISCVVector::BI__builtin_rvv_vssra_vv: + case RISCVVector::BI__builtin_rvv_vssra_vx: + case RISCVVector::BI__builtin_rvv_vssrl_vv: + case RISCVVector::BI__builtin_rvv_vssrl_vx: + case RISCVVector::BI__builtin_rvv_vnclip_wv: + case RISCVVector::BI__builtin_rvv_vnclip_wx: + case RISCVVector::BI__builtin_rvv_vnclipu_wv: + case RISCVVector::BI__builtin_rvv_vnclipu_wx: return SemaBuiltinConstantArgRange(TheCall, 2, 0, 3); case RISCVVector::BI__builtin_rvv_vaaddu_vv_tu: case RISCVVector::BI__builtin_rvv_vaaddu_vx_tu:
Index: clang/lib/Support/RISCVVIntrinsicUtils.cpp =================================================================== --- clang/lib/Support/RISCVVIntrinsicUtils.cpp +++ clang/lib/Support/RISCVVIntrinsicUtils.cpp @@ -1080,10 +1080,9 @@ } else { if (PolicyAttrs.isTUPolicy()) appendPolicySuffix("_tu"); - else if (PolicyAttrs.isTAPolicy()) { - if (HasPolicy) - BuiltinName += "_ta"; - } else + else if (PolicyAttrs.isTAPolicy()) // no suffix needed + return; + else llvm_unreachable("Unhandled policy condition"); } } Index: clang/lib/Sema/SemaChecking.cpp =================================================================== --- clang/lib/Sema/SemaChecking.cpp +++ clang/lib/Sema/SemaChecking.cpp @@ -4697,24 +4697,24 @@ case RISCV::BI__builtin_riscv_aes64ks1i_64: return SemaBuiltinConstantArgRange(TheCall, 1, 0, 10); // Check if value range for vxrm is in [0, 3] - case RISCVVector::BI__builtin_rvv_vaaddu_vv_ta: - case RISCVVector::BI__builtin_rvv_vaaddu_vx_ta: - case RISCVVector::BI__builtin_rvv_vaadd_vv_ta: - case RISCVVector::BI__builtin_rvv_vaadd_vx_ta: - case RISCVVector::BI__builtin_rvv_vasubu_vv_ta: - case RISCVVector::BI__builtin_rvv_vasubu_vx_ta: - case RISCVVector::BI__builtin_rvv_vasub_vv_ta: - case RISCVVector::BI__builtin_rvv_vasub_vx_ta: - case RISCVVector::BI__builtin_rvv_vsmul_vv_ta: - case RISCVVector::BI__builtin_rvv_vsmul_vx_ta: - case RISCVVector::BI__builtin_rvv_vssra_vv_ta: - case RISCVVector::BI__builtin_rvv_vssra_vx_ta: - case RISCVVector::BI__builtin_rvv_vssrl_vv_ta: - case RISCVVector::BI__builtin_rvv_vssrl_vx_ta: - case RISCVVector::BI__builtin_rvv_vnclip_wv_ta: - case RISCVVector::BI__builtin_rvv_vnclip_wx_ta: - case RISCVVector::BI__builtin_rvv_vnclipu_wv_ta: - case RISCVVector::BI__builtin_rvv_vnclipu_wx_ta: + case RISCVVector::BI__builtin_rvv_vaaddu_vv: + case RISCVVector::BI__builtin_rvv_vaaddu_vx: + case RISCVVector::BI__builtin_rvv_vaadd_vv: + case RISCVVector::BI__builtin_rvv_vaadd_vx: + case RISCVVector::BI__builtin_rvv_vasubu_vv: + case RISCVVector::BI__builtin_rvv_vasubu_vx: + case RISCVVector::BI__builtin_rvv_vasub_vv: + case RISCVVector::BI__builtin_rvv_vasub_vx: + case RISCVVector::BI__builtin_rvv_vsmul_vv: + case RISCVVector::BI__builtin_rvv_vsmul_vx: + case RISCVVector::BI__builtin_rvv_vssra_vv: + case RISCVVector::BI__builtin_rvv_vssra_vx: + case RISCVVector::BI__builtin_rvv_vssrl_vv: + case RISCVVector::BI__builtin_rvv_vssrl_vx: + case RISCVVector::BI__builtin_rvv_vnclip_wv: + case RISCVVector::BI__builtin_rvv_vnclip_wx: + case RISCVVector::BI__builtin_rvv_vnclipu_wv: + case RISCVVector::BI__builtin_rvv_vnclipu_wx: return SemaBuiltinConstantArgRange(TheCall, 2, 0, 3); case RISCVVector::BI__builtin_rvv_vaaddu_vv_tu: case RISCVVector::BI__builtin_rvv_vaaddu_vx_tu:
_______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits