================
@@ -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
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to