[PATCH] D48778: [llvm-exegesis] Change how the native architecture is determined

[John Brawn via Phabricator via llvm-commits]

john.brawn added inline comments.


================
Comment at: tools/llvm-exegesis/CMakeLists.txt:14
 # Link the native exegesis target if compiled and on the right host.
-if ((LLVM_TARGETS_TO_BUILD MATCHES "X86") AND ("${LLVM_NATIVE_ARCH}" STREQUAL "X86"))
-  set(LLVM_EXEGESIS_NATIVE_ARCH "X86")
+if ((LLVM_TARGETS_TO_BUILD MATCHES "${LLVM_NATIVE_ARCH}") AND (LLVM_EXEGESIS_TARGETS MATCHES "${LLVM_NATIVE_ARCH}"))
+  set(LLVM_EXEGESIS_NATIVE_ARCH "${LLVM_NATIVE_ARCH}")
----------------
courbet wrote:
> Does this work if the variable LLVM_EXEGESIS_TARGETS is never set (e.g. if llvm targets to build is just "ARM") ?
Yes, we get the some behaviour as we currently do (LLVM_EXEGESIS_NATIVE_ARCH is not set, so the default exegesis target is used).


Repository:
  rL LLVM

https://reviews.llvm.org/D48778