================
@@ -439,6 +439,13 @@ ExprResult Sema::ActOnBuiltinBitCastExpr(SourceLocation 
KWLoc, Declarator &D,
 ExprResult Sema::BuildBuiltinBitCastExpr(SourceLocation KWLoc,
                                          TypeSourceInfo *TSI, Expr *Operand,
                                          SourceLocation RParenLoc) {
+  if (Operand->hasPlaceholderType()) {
----------------
shafik wrote:

@cor3ntin I see you asked for the check to be removed on `Operand` and I can 
see in `ParseBuiltinBitCast` we do indeed check the type is valid but I think 
we should have added an asseet: `assert(Operand && "Operand should not be a 
nullptr");` or something similar b/c this seems like an invariant here. 

https://github.com/llvm/llvm-project/pull/200574
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to