[PATCH] D142016: [Clang][RISCV] Simplify RVV intrinsic policy suffix

[Yueh-Ting (eop) Chen via Phabricator via cfe-commits]

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