[clang] 5ca710a - Revert "Silence unused variable warning in NDEBUG builds"

Mitch Phillips via cfe-commits cfe-commits at lists.llvm.org
Mon Mar 27 05:02:58 PDT 2023


Author: Mitch Phillips
Date: 2023-03-27T05:01:53-07:00
New Revision: 5ca710ab148b0815c2b7b03fe2af643e637bbc7d

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

LOG: Revert "Silence unused variable warning in NDEBUG builds"

This reverts commit 8c7c1f11ffaacf762e612c65440fd2cbb58ee426.

Reason: Dependent change https://reviews.llvm.org/D126959 broke the ASan
buildbots. See that phabricator review for more comments.

Added: 
    

Modified: 
    clang/lib/Lex/ModuleMap.cpp

Removed: 
    


################################################################################
diff  --git a/clang/lib/Lex/ModuleMap.cpp b/clang/lib/Lex/ModuleMap.cpp
index 44c872336ce9c..f2b2d0b8c69f1 100644
--- a/clang/lib/Lex/ModuleMap.cpp
+++ b/clang/lib/Lex/ModuleMap.cpp
@@ -936,7 +936,6 @@ Module *ModuleMap::createModuleForImplementationUnit(SourceLocation Loc,
   // Mark the main source file as being within the newly-created module so that
   // declarations and macros are properly visibility-restricted to it.
   auto *MainFile = SourceMgr.getFileEntryForID(SourceMgr.getMainFileID());
-  (void)MainFile;
   assert(MainFile && "no input file for module implementation");
 
   return Result;


        


More information about the cfe-commits mailing list