Placeholder c-strings don't need to be instance variables.
|21–22 ↗||(On Diff #74617)|
I would like to use an anonymous namespace but the LLVM coding standards say
I don't really buy their reasoning, but we are supposed to be following their style now aren't we?
kUnresolvedValue/kUnresolvedCompositeValue does sound better, I'll rename them.
|116 ↗||(On Diff #74622)|
After couple of minutes I see the lack of logic there: the value is still composite but we return kUnresolvedValue instead of kUnresolvedCompositeValue.
I think we have to rename kUnresolvedValue back to kUnknownValue.