xplat: do not inline class function in source file

- Fixes clang/homebrew '-Os' build breaks
- Remove unused definition from CMakeLists

CMakeLists.txt

@@ -90,7 +90,6 @@ if(CC_EMBED_ICU_SH)
     unset(CC_EMBED_ICU_SH CACHE)
     set(CC_EMBED_ICU 1)
     set(ICU_INCLUDE_PATH "deps/icu/source/output/include/")
-    add_definitions(-DU_STATIC_IMPLEMENTATION)
 endif()
 
 if(ICU_INCLUDE_PATH_SH)

lib/Backend/IRBuilderAsmJs.cpp

@@ -943,7 +943,7 @@ IRBuilderAsmJs::BuildArgInTracing()
     int32 doubleArgInCount = 0;
     int32 simd128ArgInCount = 0;
 
-    Js::ArgSlot nArgs = 0; 
+    Js::ArgSlot nArgs = 0;
     if (m_func->GetJITFunctionBody()->HasImplicitArgIns())
     {
         // -1 to remove the implicit this pointer
@@ -3260,7 +3260,7 @@ IRBuilderAsmJs::BuildLong1Double1(Js::OpCodeAsmJs newOpcode, uint32 offset, Js::
 {
     IR::RegOpnd * srcOpnd = BuildSrcOpnd(src1RegSlot, TyFloat64);
     srcOpnd->SetValueType(ValueType::Float);
-    
+
     IRType dstType;
     Js::OpCode op;
     bool doTruncTrapCheck = false;
@@ -3536,7 +3536,7 @@ IR::Instr* IRBuilderAsmJs::GenerateStSlotForReturn(IR::RegOpnd* srcOpnd, IRType
     return stSlotInstr;
 }
 
-inline Js::OpCode IRBuilderAsmJs::GetSimdOpcode(Js::OpCodeAsmJs asmjsOpcode)
+Js::OpCode IRBuilderAsmJs::GetSimdOpcode(Js::OpCodeAsmJs asmjsOpcode)
 {
     Js::OpCode opcode = (Js::OpCode) 0;
     Assert(IsSimd128AsmJsOpcode(asmjsOpcode));
@@ -5505,7 +5505,7 @@ IRBuilderAsmJs::BuildUint16x8_1Uint8x16_1(Js::OpCodeAsmJs newOpcode, uint32 offs
 void IRBuilderAsmJs::BuildUint8x16_1Int16(Js::OpCodeAsmJs newOpcode, uint32 offset, BUILD_SIMD_ARGS_REG17)
 {
     AssertMsg(newOpcode == Js::OpCodeAsmJs::Simd128_IntsToU16, "Unexpected opcode for this format.");
- 
+
     uint const LANES = 16;
     Js::RegSlot srcRegSlots[LANES];
 

lib/Backend/InterpreterThunkEmitter.cpp

@@ -304,7 +304,7 @@ void InterpreterThunkEmitter::NewThunkBlock()
     BYTE* buffer;
 
     EmitBufferAllocation<VirtualAllocWrapper, PreReservedVirtualAllocWrapper> * allocation = emitBufferManager.AllocateBuffer(BlockSize, &buffer);
-    if (allocation == nullptr) 
+    if (allocation == nullptr)
     {
         Js::Throw::OutOfMemory();
     }
@@ -683,9 +683,7 @@ void InterpreterThunkEmitter::EncodeInterpreterThunk(
 }
 #endif
 
-
-
-inline /*static*/
+/*static*/
 DWORD InterpreterThunkEmitter::FillDebugBreak(_Out_writes_bytes_all_(count) BYTE* dest, _In_ DWORD count)
 {
 #if defined(_M_ARM)
@@ -697,9 +695,7 @@ DWORD InterpreterThunkEmitter::FillDebugBreak(_Out_writes_bytes_all_(count) BYTE
     return count;
 }
 
-
-
-inline /*static*/
+/*static*/
 DWORD InterpreterThunkEmitter::CopyWithAlignment(
     _Out_writes_bytes_all_(sizeInBytes) BYTE* dest,
     _In_ const DWORD sizeInBytes,
@@ -876,4 +872,3 @@ bool ThunkBlock::IsFreeListEmpty() const
 }
 
 #endif
-

lib/Parser/CharSet.cpp

@@ -196,7 +196,7 @@ namespace UnifiedRegex
     // CharSetNode
     // ----------------------------------------------------------------------
 
-    inline CharSetNode* CharSetNode::For(ArenaAllocator* allocator, int level)
+    CharSetNode* CharSetNode::For(ArenaAllocator* allocator, int level)
     {
         if (level == 0)
             return Anew(allocator, CharSetLeaf);

lib/Runtime/Library/JavascriptString.cpp

@@ -403,7 +403,7 @@ case_2:
         }
     }
 
-    inline JavascriptString* JavascriptString::ConcatDestructive(JavascriptString* pstRight)
+    JavascriptString* JavascriptString::ConcatDestructive(JavascriptString* pstRight)
     {
         Assert(pstRight);