Author: eopXD Date: 2023-01-24T02:25:25-08:00 New Revision: 5a1c6ec1041f19806f3e94c3cb750b9072087c14
URL: https://github.com/llvm/llvm-project/commit/5a1c6ec1041f19806f3e94c3cb750b9072087c14 DIFF: https://github.com/llvm/llvm-project/commit/5a1c6ec1041f19806f3e94c3cb750b9072087c14.diff LOG: [8/15][Clang][RISCV][NFC] Always emit PolicyAttr in riscv_vector_builtin_cg.inc The extra logic here is redundant. Also sneaked in an one-liner for emitting `IsMasked`. This is the 8th commit of a patch-set that aims to change the default policy for RVV intrinsics from TAMU to TAMA. Please refer to the cover letter in the 1st commit (D141573) for an overview. Reviewed By: kito-cheng Differential Revision: https://reviews.llvm.org/D141762 Added: Modified: clang/utils/TableGen/RISCVVEmitter.cpp Removed: ################################################################################ diff --git a/clang/utils/TableGen/RISCVVEmitter.cpp b/clang/utils/TableGen/RISCVVEmitter.cpp index 59a250aa29c9..39925cc5c24b 100644 --- a/clang/utils/TableGen/RISCVVEmitter.cpp +++ b/clang/utils/TableGen/RISCVVEmitter.cpp @@ -163,18 +163,11 @@ void emitCodeGenSwitchBody(const RVVIntrinsic *RVVI, raw_ostream &OS) { OS << " ID = Intrinsic::riscv_" + RVVI->getIRName() + ";\n"; if (RVVI->getNF() >= 2) OS << " NF = " + utostr(RVVI->getNF()) + ";\n"; - // We had initialized PolicyAttrs as TU/TUMU in CodeGen function. - if (!RVVI->getPolicyAttrs().isTUPolicy() && - !RVVI->getPolicyAttrs().isTUMUPolicy() && !RVVI->hasPassthruOperand() && - !RVVI->hasManualCodegen() && RVVI->hasVL()) - OS << " PolicyAttrs = " << RVVI->getPolicyAttrsBits() << ";\n"; + + OS << " PolicyAttrs = " << RVVI->getPolicyAttrsBits() << ";\n"; if (RVVI->hasManualCodegen()) { - OS << " PolicyAttrs = " << RVVI->getPolicyAttrsBits() << ";\n"; - if (RVVI->isMasked()) - OS << "IsMasked = true;\n"; - else - OS << "IsMasked = false;\n"; + OS << "IsMasked = " << (RVVI->isMasked() ? "true" : "false") << ";\n"; OS << RVVI->getManualCodegen(); OS << "break;\n"; return; _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
