[PATCH] D77741: [FileCheck] Better diagnostic for format conflict

[Joel E. Denny via Phabricator via llvm-commits]

jdenny added inline comments.


================
Comment at: llvm/lib/Support/FileCheck.cpp:434
     Format = ExplicitFormat;
-  else if (ImplicitFormat == ExpressionFormat::Kind::Conflict)
-    return ErrorDiagnostic::get(
-        SM, UseExpr,
-        "variables with conflicting format specifier: need an explicit one");
-  else if (bool(ImplicitFormat))
-    Format = ImplicitFormat;
-  else
+  else if (ExpressionASTPointer) {
+    Expected<ExpressionFormat> ImplicitFormat =
----------------
jdenny wrote:
> Is there a logic change here for the case of !ExpressionASTPointer?
> 
> That used to mean ImplicitFormat = NoFormat and then Format = ImplicitFormat.
> 
> Now it means Format = Unsigned.
> 
> Did I read that correctly?  Why the change?
I don't see where this was addressed in your update.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D77741/new/

https://reviews.llvm.org/D77741