[clang] [clang][bytecode] Stop relying on `CheckEvaluationResult()` (PR #186045)

[Erich Keane via cfe-commits]

================
@@ -5057,8 +5057,17 @@ const Function *Compiler<Emitter>::getFunction(const FunctionDecl *FD) {
 
 template <class Emitter>
 bool Compiler<Emitter>::visitExpr(const Expr *E, bool DestroyToplevelScope) {
+  if (E->getType().isNull())
----------------
erichkeane wrote:

When do we have an expression that itself is valid, but with a null type? 

https://github.com/llvm/llvm-project/pull/186045