This is an archive of the discontinued LLVM Phabricator instance.

Paths

[llvm-exegesis] Fix PR39021.
ClosedPublic

Authored by courbet on Sep 21 2018, 1:28 AM.

Reviewers

Commits

Summary

The set statements was incorrectly reading the value of the local variable and
setting the value of the parent variable.

Repository

rL LLVM

Build Status

Buildable 22910
Build 22910: arc lint + arc unit

This revision is now accepted and ready to land.Sep 21 2018, 5:33 AM

This was committed as r342865.

Path

Size

tools/

llvm-exegesis/

lib/

8 lines

				set(TARGETS_TO_APPEND "")

	if (LLVM_TARGETS_TO_BUILD MATCHES "X86")			if (LLVM_TARGETS_TO_BUILD MATCHES "X86")
	add_subdirectory(X86)			add_subdirectory(X86)
	set(LLVM_EXEGESIS_TARGETS "${LLVM_EXEGESIS_TARGETS} X86" PARENT_SCOPE)			set(TARGETS_TO_APPEND "${TARGETS_TO_APPEND} X86")
	endif()			endif()
	if (LLVM_TARGETS_TO_BUILD MATCHES "AArch64")			if (LLVM_TARGETS_TO_BUILD MATCHES "AArch64")
	add_subdirectory(AArch64)			add_subdirectory(AArch64)
	set(LLVM_EXEGESIS_TARGETS "${LLVM_EXEGESIS_TARGETS} AArch64" PARENT_SCOPE)			set(TARGETS_TO_APPEND "${TARGETS_TO_APPEND} AArch64")
	endif()			endif()

				set(LLVM_EXEGESIS_TARGETS "${LLVM_EXEGESIS_TARGETS} ${TARGETS_TO_APPEND}" PARENT_SCOPE)

	add_library(LLVMExegesis			add_library(LLVMExegesis
	STATIC			STATIC
	Analysis.cpp			Analysis.cpp
	Assembler.cpp			Assembler.cpp
	BenchmarkResult.cpp			BenchmarkResult.cpp
	BenchmarkRunner.cpp			BenchmarkRunner.cpp
	Clustering.cpp			Clustering.cpp
	Latency.cpp			Latency.cpp
	Show All 31 Lines