eopXD added inline comments.
================ Comment at: clang/lib/Support/RISCVVIntrinsicUtils.cpp:977 +llvm::SmallVector<Policy> RVVIntrinsic::getSupportedUnMaskedPolicies() { + return { ---------------- craig.topper wrote: > Possible improvement for a different patch: > > Could these policy lists be static const arrays and this function could > returned an ArrayRef pointing to the appropriate list instead constructing a > SmallVector? Yes. I will use a flyweight pattern to simplify things here. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D141573/new/ https://reviews.llvm.org/D141573 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits