aaron.ballman added a comment.

I guess I'm not seeing much motivation for this change. We already have 
`clang::getNullabilitySpelling()` and `const StreamingDiagnostic 
&clang::operator<<(const StreamingDiagnostic &DB, DiagNullabilityKind 
nullability)` and now we're adding a third way to get this information. If this 
is just for debug/testing purposes, can we use existing debug formatters to 
convert the enumeration value into the enumerator name instead?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D149650

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to