[PATCH] D142016: [Clang][RISCV] Simplify RVV intrinsic policy suffix
Yueh-Ting (eop) Chen via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Jan 18 07:00:55 PST 2023
eopXD added inline comments.
================
Comment at: clang/lib/Support/RISCVVIntrinsicUtils.cpp:875
- updateNamesAndPolicy(IsMasked, hasPolicy(), HasTailPolicy, HasMaskPolicy,
- Name, BuiltinName, OverloadedName, PolicyAttrs);
----------------
@kito-cheng I think the removal of `HasTailPolicy` and `HasMaskPolicy` here and under `SemaRISCVVectorLookup.cpp`, `RISCVVEmitter.cpp` should resolve your concern on these two attributes blending into the `Policy` object.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D142016/new/
https://reviews.llvm.org/D142016
More information about the cfe-commits
mailing list