This is an archive of the discontinued LLVM Phabricator instance.

Set -std=c99 flag on Darwin for builtins too
AcceptedPublic

Authored by angelsl on Oct 20 2015, 8:00 PM.

Reviewers

Summary

Although this isn't really needed, but I guess for consistency's sake.

angelsl retitled this revision from to Set -std=c99 flag on Darwin for builtins too.

angelsl updated this object.

angelsl added a reviewer: compnerd.

Would appreciate if someone took a look at this. Thanks.

Would appreciate it if someone reviewed this.

LGTM.

This revision is now accepted and ready to land.Nov 20 2015, 10:49 AM

Please help me commit this. Thanks.

Looks like patch was not committed.

Path

Size

cmake/

Modules/

3 lines

Context not available.
	set(CMAKE_C_FLAGS "-fPIC -fvisibility=hidden -DVISIBILITY_HIDDEN -Wall -fomit-frame-pointer")	set(CMAKE_C_FLAGS "-fPIC -fvisibility=hidden -DVISIBILITY_HIDDEN -Wall -fomit-frame-pointer")
	set(CMAKE_CXX_FLAGS ${CMAKE_C_FLAGS})	set(CMAKE_CXX_FLAGS ${CMAKE_C_FLAGS})
	set(CMAKE_ASM_FLAGS ${CMAKE_C_FLAGS})	set(CMAKE_ASM_FLAGS ${CMAKE_C_FLAGS})
		append_string_if(COMPILER_RT_HAS_STD_C99_FLAG -std=c99 maybe_stdc99)

	set(PROFILE_SOURCES ../profile/InstrProfiling	set(PROFILE_SOURCES ../profile/InstrProfiling
	../profile/InstrProfilingBuffer	../profile/InstrProfilingBuffer
Context not available.
	OS ${os}	OS ${os}
	ARCH ${arch}	ARCH ${arch}
	SOURCES ${filtered_sources}	SOURCES ${filtered_sources}
	CFLAGS -arch ${arch}	CFLAGS -arch ${arch} ${maybe_stdc99}
	PARENT_TARGET builtins)	PARENT_TARGET builtins)
	endforeach()	endforeach()

Context not available.