From 322fcfee34effec55c20a4986f19a3894a132284 Mon Sep 17 00:00:00 2001 From: Rui Ueyama Date: Mon, 22 Jan 2018 23:27:50 +0000 Subject: Revert r322595: Specify inline for isWhitespace in CommandLine.cpp The original change was made based on a misunderstanding that -DCMAKE_BUILD_TYPE=RelWithDebugInfo would produce the same executable as -DCMAKE_BUILD_TYPE=Release modulo debug info. Turned out that's not true -- it at least disables some optimizations such as function inlining. llvm-svn: 323161 --- llvm/lib/Support/CommandLine.cpp | 2 +- 1 file changed, 1 insertion(+), 1 deletion(-) (limited to 'llvm/lib/Support/CommandLine.cpp') diff --git a/llvm/lib/Support/CommandLine.cpp b/llvm/lib/Support/CommandLine.cpp index 451c3f46036..d95b791972c 100644 --- a/llvm/lib/Support/CommandLine.cpp +++ b/llvm/lib/Support/CommandLine.cpp @@ -688,7 +688,7 @@ static bool EatsUnboundedNumberOfValues(const Option *O) { O->getNumOccurrencesFlag() == cl::OneOrMore; } -static inline bool isWhitespace(char C) { +static bool isWhitespace(char C) { return C == ' ' || C == '\t' || C == '\r' || C == '\n'; } -- cgit v1.2.3