From 31b01d59cc46942d4bdbfa18b108b54048703e83 Mon Sep 17 00:00:00 2001 From: Nico Weber Date: Tue, 19 Jan 2016 22:46:33 +0000 Subject: [PATCH] Reenable -Wexpansion-to-defined. I think I fixed all instances of this in the codebase (r258202, 258200, 258190). Also, the suppression didn't have an effect on bots using make anyways, and it looks like many bots still use configure/make bots. llvm-svn: 258210 --- cmake/modules/HandleLLVMOptions.cmake | 3 --- 1 file changed, 3 deletions(-) diff --git a/cmake/modules/HandleLLVMOptions.cmake b/cmake/modules/HandleLLVMOptions.cmake index c946b791cb4..6db258ff66a 100644 --- a/cmake/modules/HandleLLVMOptions.cmake +++ b/cmake/modules/HandleLLVMOptions.cmake @@ -403,9 +403,6 @@ elseif( LLVM_COMPILER_IS_GCC_COMPATIBLE ) append("-Wall -W -Wno-unused-parameter -Wwrite-strings" CMAKE_C_FLAGS CMAKE_CXX_FLAGS) append("-Wcast-qual" CMAKE_CXX_FLAGS) - # FIXME: Clean up the codebase, then remove this flag. - add_flag_if_supported("-Wno-expansion-to-defined" EXPANSION_TO_DEFINED_FLAG) - # Turn off missing field initializer warnings for gcc to avoid noise from # false positives with empty {}. Turn them on otherwise (they're off by # default for clang).