[clang][dataflow] Debug string for value kinds.

[xazax.hun]

clang/lib/Analysis/FlowSensitive/DebugSupport.cpp

34

I think all of these debugString functions could return a llvm::StringRef or an std::string_view.

[wyt]

Use llvm::StringRef for returning static strings.

clang/include/clang/Analysis/FlowSensitive/DebugSupport.h

 
 #include "clang/Analysis/FlowSensitive/Solver.h"
 #include "clang/Analysis/FlowSensitive/Value.h"
 #include "llvm/ADT/DenseMap.h"
 
 namespace clang {
 namespace dataflow {
 
+/// Returns a string representation of a value kind.
+std::string debugString(Value::Kind Kind);
+
 /// Returns a string representation of a boolean assignment to true or false.
 std::string debugString(Solver::Result::Assignment Assignment);
 
 /// Returns a string representation of the result status of a SAT check.
 std::string debugString(Solver::Result::Status Status);
 
 /// Returns a string representation for the boolean value `B`.
 ///

clang/lib/Analysis/FlowSensitive/DebugSupport.cpp

 
 namespace clang {
 namespace dataflow {
 
 using llvm::AlignStyle;
 using llvm::fmt_pad;
 using llvm::formatv;
 
+std::string debugString(Value::Kind Kind) {
+  switch (Kind) {
+  case Value::Kind::Integer:
+    return "Integer";
+  case Value::Kind::Reference:
+    return "Reference";
+  case Value::Kind::Pointer:
+    return "Pointer";
+  case Value::Kind::Struct:
+    return "Struct";
+  case Value::Kind::AtomicBool:
+    return "AtomicBool";
+  case Value::Kind::Conjunction:
+    return "Conjunction";
+  case Value::Kind::Disjunction:
+    return "Disjunction";
+  case Value::Kind::Negation:
+    return "Negation";
+  case Value::Kind::Implication:
+    return "Implication";
+  case Value::Kind::Biconditional:
+    return "Biconditional";
+  }
+  llvm_unreachable("Unhandled value kind");
+}
+
 std::string debugString(Solver::Result::Assignment Assignment) {
   switch (Assignment) {
   case Solver::Result::Assignment::AssignedFalse:
     return "False";
   case Solver::Result::Assignment::AssignedTrue:
     return "True";
   }
   llvm_unreachable("Booleans can only be assigned true/false");