[PATCH] D125893: [RISCV][NFC] Change interface of RVVIntrinsic::getSuffixStr
Kito Cheng via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed May 18 08:25:51 PDT 2022
kito-cheng updated this revision to Diff 430390.
kito-cheng added a comment.
Changes:
- Fix dumb typo...
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D125893/new/
https://reviews.llvm.org/D125893
Files:
clang/include/clang/Support/RISCVVIntrinsicUtils.h
clang/lib/Support/RISCVVIntrinsicUtils.cpp
Index: clang/lib/Support/RISCVVIntrinsicUtils.cpp
===================================================================
--- clang/lib/Support/RISCVVIntrinsicUtils.cpp
+++ clang/lib/Support/RISCVVIntrinsicUtils.cpp
@@ -928,7 +928,7 @@
std::string RVVIntrinsic::getSuffixStr(
BasicType Type, int Log2LMUL,
- const llvm::SmallVector<PrototypeDescriptor> &PrototypeDescriptors) {
+ const llvm::ArrayRef<PrototypeDescriptor> &PrototypeDescriptors) {
SmallVector<std::string> SuffixStrs;
for (auto PD : PrototypeDescriptors) {
auto T = RVVType::computeType(Type, Log2LMUL, PD);
Index: clang/include/clang/Support/RISCVVIntrinsicUtils.h
===================================================================
--- clang/include/clang/Support/RISCVVIntrinsicUtils.h
+++ clang/include/clang/Support/RISCVVIntrinsicUtils.h
@@ -305,13 +305,15 @@
unsigned NF = 1;
public:
- RVVIntrinsic(llvm::StringRef Name, llvm::StringRef Suffix, llvm::StringRef MangledName,
- llvm::StringRef MangledSuffix, llvm::StringRef IRName, bool IsMasked,
- bool HasMaskedOffOperand, bool HasVL, PolicyScheme Scheme,
- bool HasUnMaskedOverloaded, bool HasBuiltinAlias,
- llvm::StringRef ManualCodegen, const RVVTypes &Types,
+ RVVIntrinsic(llvm::StringRef Name, llvm::StringRef Suffix,
+ llvm::StringRef MangledName, llvm::StringRef MangledSuffix,
+ llvm::StringRef IRName, bool IsMasked, bool HasMaskedOffOperand,
+ bool HasVL, PolicyScheme Scheme, bool HasUnMaskedOverloaded,
+ bool HasBuiltinAlias, llvm::StringRef ManualCodegen,
+ const RVVTypes &Types,
const std::vector<int64_t> &IntrinsicTypes,
- const std::vector<llvm::StringRef> &RequiredFeatures, unsigned NF);
+ const std::vector<llvm::StringRef> &RequiredFeatures,
+ unsigned NF);
~RVVIntrinsic() = default;
RVVTypePtr getOutputType() const { return OutputType; }
@@ -341,9 +343,9 @@
// Return the type string for a BUILTIN() macro in Builtins.def.
std::string getBuiltinTypeStr() const;
- static std::string getSuffixStr(
- BasicType Type, int Log2LMUL,
- const llvm::SmallVector<PrototypeDescriptor> &PrototypeDescriptors);
+ static std::string
+ getSuffixStr(BasicType Type, int Log2LMUL,
+ const llvm::ArrayRef<PrototypeDescriptor> &PrototypeDescriptors);
};
} // end namespace RISCV
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D125893.430390.patch
Type: text/x-patch
Size: 2474 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20220518/00c41062/attachment-0001.bin>
More information about the cfe-commits
mailing list