[PATCH] D63139: [Diagnostics] Implement -Wswitch-unreachable

[John McCall via Phabricator via cfe-commits]

rjmccall added a comment.

I agree that we really shouldn't use most of those — we shouldn't have null types or null child expressions anywhere in the AST.  (Accessors might return null for *optional* children, of course, but that's different.)  And yeah, a big part of that would be having an error type that could be used in various places (instead of e.g. defaulting to `int`).


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

https://reviews.llvm.org/D63139