This is an archive of the discontinued LLVM Phabricator instance.

[libcxx] Fix C1XX implementation of DoNotOptimize
ClosedPublic

Authored by CaseyCarter on Apr 25 2017, 4:25 PM.

Reviewers

mclow.lists
EricWF

Commits

Summary

...in test_macros.h

Oops. Thanks for the fix.

This revision is now accepted and ready to land.May 4 2017, 1:16 AM

This revision was automatically updated to reflect the committed changes.

Path

Size

libcxx/

trunk/

test/

support/

3 lines

	Show First 20 Lines • Show All 203 Lines • ▼ Show 20 Lines
	template <class Tp>			template <class Tp>
	inline void DoNotOptimize(Tp const& value) {			inline void DoNotOptimize(Tp const& value) {
	asm volatile("" : : "g"(value) : "memory");			asm volatile("" : : "g"(value) : "memory");
	}			}
	#else			#else
	#include <intrin.h>			#include <intrin.h>
	template <class Tp>			template <class Tp>
	inline void DoNotOptimize(Tp const& value) {			inline void DoNotOptimize(Tp const& value) {
	const volatile void* volatile = __builtin_addressof(value);			const volatile void* volatile unused = __builtin_addressof(value);
				static_cast<void>(unused);
	_ReadWriteBarrier();			_ReadWriteBarrier();
	}			}
	#endif			#endif

	#if defined(__GNUC__)			#if defined(__GNUC__)
	#pragma GCC diagnostic pop			#pragma GCC diagnostic pop
	#endif			#endif

	#endif // SUPPORT_TEST_MACROS_HPP			#endif // SUPPORT_TEST_MACROS_HPP