[clang] [llvm] [mlir] [Support] Validate number of arguments passed to formatv() (PR #105745)
Rahul Joshi via cfe-commits
cfe-commits at lists.llvm.org
Thu Aug 29 07:51:31 PDT 2024
================
@@ -1654,12 +1654,12 @@ void OperationFormat::genElementParser(FormatElement *element, MethodBody &body,
dir->shouldBeQualified() ? qualifiedTypeParserCode : typeParserCode;
TypeSwitch<FormatElement *>(dir->getArg())
.Case<OperandVariable, ResultVariable>([&](auto operand) {
- body << formatv(parserCode,
+ body << formatv(false, parserCode,
----------------
jurahul wrote:
Yeah, because parserCode is either with 1 or 2 placeholders:
```
const char *parserCode =
dir->shouldBeQualified() ? qualifiedTypeParserCode : typeParserCode;
```
We could add if's in the code around calls to formatv() as well, but I though the code duplication just to get validation was not warranted.
https://github.com/llvm/llvm-project/pull/105745
More information about the cfe-commits
mailing list