[clang] a69975d - [clang][bytecode] Remove unnecessary if statement (#126223)

via cfe-commits cfe-commits at lists.llvm.org
Fri Feb 7 05:24:19 PST 2025


Author: Timm Baeder
Date: 2025-02-07T14:24:15+01:00
New Revision: a69975d8349c1e0dcb86814df67d7160395b468e

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

LOG: [clang][bytecode] Remove unnecessary if statement (#126223)

This doesn't seem to do any good and breaks a few libc++ tests. Remove
the special case.

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 1e1e96a1c4782f4..1f0e022edcd7687 100644
--- a/clang/lib/AST/ByteCode/Compiler.cpp
+++ b/clang/lib/AST/ByteCode/Compiler.cpp
@@ -6234,9 +6234,6 @@ bool Compiler<Emitter>::visitDeclRef(const ValueDecl *D, const Expr *E) {
 
       return this->emitGetPtrParam(It->second.Offset, E);
     }
-
-    if (D->getType()->isReferenceType())
-      return this->emitDummyPtr(D, E);
   }
 
   // In case we need to re-visit a declaration.


        


More information about the cfe-commits mailing list