[clang] 38b8e54 - [clang][bytecode][NFC] Remove containsErrors() check from delegate (#105804)

via cfe-commits cfe-commits at lists.llvm.org
Fri Aug 23 02:55:34 PDT 2024


Author: Timm Baeder
Date: 2024-08-23T11:55:30+02:00
New Revision: 38b8e54682567d685bc03f9fdef26baa6b708ef9

URL: https://github.com/llvm/llvm-project/commit/38b8e54682567d685bc03f9fdef26baa6b708ef9
DIFF: https://github.com/llvm/llvm-project/commit/38b8e54682567d685bc03f9fdef26baa6b708ef9.diff

LOG: [clang][bytecode][NFC] Remove containsErrors() check from delegate (#105804)

This check was removed a while ago from visit(), remove it from
delegate() as well.

Added: 
    

Modified: 
    clang/lib/AST/ByteCode/Compiler.cpp

Removed: 
    


################################################################################
diff  --git a/clang/lib/AST/ByteCode/Compiler.cpp b/clang/lib/AST/ByteCode/Compiler.cpp
index 655983a1ca0494..d24384ca4ac7a4 100644
--- a/clang/lib/AST/ByteCode/Compiler.cpp
+++ b/clang/lib/AST/ByteCode/Compiler.cpp
@@ -3262,9 +3262,6 @@ template <class Emitter> bool Compiler<Emitter>::discard(const Expr *E) {
 }
 
 template <class Emitter> bool Compiler<Emitter>::delegate(const Expr *E) {
-  if (E->containsErrors())
-    return this->emitError(E);
-
   // We're basically doing:
   // OptionScope<Emitter> Scope(this, DicardResult, Initializing);
   // but that's unnecessary of course.


        


More information about the cfe-commits mailing list