<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40">
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
<meta name="Generator" content="Microsoft Word 14 (filtered medium)">
<style><!--
/* Font Definitions */
@font-face
        {font-family:Calibri;
        panose-1:2 15 5 2 2 2 4 3 2 4;}
@font-face
        {font-family:Tahoma;
        panose-1:2 11 6 4 3 5 4 4 2 4;}
/* Style Definitions */
p.MsoNormal, li.MsoNormal, div.MsoNormal
        {margin:0cm;
        margin-bottom:.0001pt;
        font-size:12.0pt;
        font-family:"Times New Roman","serif";}
a:link, span.MsoHyperlink
        {mso-style-priority:99;
        color:blue;
        text-decoration:underline;}
a:visited, span.MsoHyperlinkFollowed
        {mso-style-priority:99;
        color:purple;
        text-decoration:underline;}
span.EmailStyle17
        {mso-style-type:personal-reply;
        font-family:"Calibri","sans-serif";
        color:#1F497D;}
.MsoChpDefault
        {mso-style-type:export-only;
        font-family:"Calibri","sans-serif";
        mso-fareast-language:EN-US;}
@page WordSection1
        {size:612.0pt 792.0pt;
        margin:72.0pt 72.0pt 72.0pt 72.0pt;}
div.WordSection1
        {page:WordSection1;}
--></style><!--[if gte mso 9]><xml>
<o:shapedefaults v:ext="edit" spidmax="1026" />
</xml><![endif]--><!--[if gte mso 9]><xml>
<o:shapelayout v:ext="edit">
<o:idmap v:ext="edit" data="1" />
</o:shapelayout></xml><![endif]-->
</head>
<body lang="EN-GB" link="blue" vlink="purple">
<div class="WordSection1">
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D">What’s happening is that<o:p></o:p></span></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D"> 
</span>set(LLVM_EXEGESIS_TARGETS "${LLVM_EXEGESIS_TARGETS} X86" PARENT_SCOPE)<o:p></o:p></p>
<p class="MsoNormal">and<o:p></o:p></p>
<p class="MsoNormal">set(LLVM_EXEGESIS_TARGETS "${LLVM_EXEGESIS_TARGETS} AArch64" PARENT_SCOPE)<o:p></o:p></p>
<p class="MsoNormal">set LLVM_EXEGESIS_TARGETS in the parent scope and not the local scope, but read from the local scope.<o:p></o:p></p>
<p class="MsoNormal">So when the first executes the local LLVM_EXEGESIS_TARGETS is unset so the parent LLVM_EXEGESIS_TARGETS<o:p></o:p></p>
<p class="MsoNormal">is set to â€œ X86”, but in the second the local LLVM_EXEGESIS_TARGETS is still unset so the parent<o:p></o:p></p>
<p class="MsoNormal">LLVM_EXEGESIS_TARGETS gets set to â€œ AArch64”.<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">As for the correct way to fix it, looking a bit at the .cmake files in llvm/cmake/modules/ it looks like the standard<o:p></o:p></p>
<p class="MsoNormal">way would be to have the file set LLVM_EXEGESIS_TARGETS locally, then in the parent. i.e. something like<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">if (LLVM_TARGETS_TO_BUILD MATCHES "X86")<o:p></o:p></p>
<p class="MsoNormal">  set(LLVM_EXEGESIS_TARGETS "${LLVM_EXEGESIS_TARGETS} X86")<o:p></o:p></p>
<p class="MsoNormal">endif()<br>
if (LLVM_TARGETS_TO_BUILD MATCHES "AArch64")<o:p></o:p></p>
<p class="MsoNormal">  set(LLVM_EXEGESIS_TARGETS "${LLVM_EXEGESIS_TARGETS} AArch64")<o:p></o:p></p>
<p class="MsoNormal">endif()<o:p></o:p></p>
<p class="MsoNormal">set(LLVM_EXEGESIS_TARGETS "${LLVM_EXEGESIS_TARGETS}" PARENT_SCOPE)<o:p></o:p></p>
<p class="MsoNormal"><o:p> </o:p></p>
<p class="MsoNormal">John<o:p></o:p></p>
<p class="MsoNormal"><span style="font-size:11.0pt;font-family:"Calibri","sans-serif";color:#1F497D"><o:p> </o:p></span></p>
<div style="border:none;border-left:solid blue 1.5pt;padding:0cm 0cm 0cm 4.0pt">
<div>
<div style="border:none;border-top:solid #B5C4DF 1.0pt;padding:3.0pt 0cm 0cm 0cm">
<p class="MsoNormal"><b><span lang="EN-US" style="font-size:10.0pt;font-family:"Tahoma","sans-serif"">From:</span></b><span lang="EN-US" style="font-size:10.0pt;font-family:"Tahoma","sans-serif""> llvm-dev [mailto:llvm-dev-bounces@lists.llvm.org]
<b>On Behalf Of </b>Steven Noonan via llvm-dev<br>
<b>Sent:</b> 20 September 2018 16:42<br>
<b>To:</b> llvm-dev@lists.llvm.org<br>
<b>Subject:</b> [llvm-dev] llvm-exegesis broken on x86<o:p></o:p></span></p>
</div>
</div>
<p class="MsoNormal"><o:p> </o:p></p>
<div>
<div>
<p class="MsoNormal">Something's up with the llvm-exegesis build process on 7.0.0 release. On my x86 machines they always complain "no exegesis target for x86_64-pc-linux-gnu, using default", and this seems to be because LLVM_EXEGESIS_INITIALIZE_NATIVE_TARGET
 never gets set by CMake.<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">The CMake scripts try to set LLVM_EXEGESIS_TARGETS in the lib subdirectory, but this doesn't seem to work properly. I made this change to figure out what's happening:<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">---<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">diff --git a/tools/llvm-exegesis/CMakeLists.txt b/tools/llvm-exegesis/CMakeLists.txt<br>
index 65b1ada8529..94842519ca7 100644<br>
--- a/tools/llvm-exegesis/CMakeLists.txt<br>
+++ b/tools/llvm-exegesis/CMakeLists.txt<br>
@@ -8,7 +8,9 @@ add_llvm_tool(llvm-exegesis<br>
   llvm-exegesis.cpp<br>
   )<br>
 <br>
+message(WARNING "LLVM_EXEGESIS_TARGETS at parent scope before: ${LLVM_EXEGESIS_TARGETS}")<br>
 add_subdirectory(lib)<br>
+message(FATAL_ERROR "LLVM_EXEGESIS_TARGETS at parent scope after: ${LLVM_EXEGESIS_TARGETS}")<br>
 <br>
 # Link the native exegesis target if compiled and on the right host.<br>
 if ((LLVM_TARGETS_TO_BUILD MATCHES "${LLVM_NATIVE_ARCH}") AND (LLVM_EXEGESIS_TARGETS MATCHES "${LLVM_NATIVE_ARCH}"))<br>
diff --git a/tools/llvm-exegesis/lib/CMakeLists.txt b/tools/llvm-exegesis/lib/CMakeLists.txt<br>
index 175c2adf9de..4dc91ef125c 100644<br>
--- a/tools/llvm-exegesis/lib/CMakeLists.txt<br>
+++ b/tools/llvm-exegesis/lib/CMakeLists.txt<br>
@@ -1,12 +1,18 @@<br>
+message( WARNING "LLVM_EXEGESIS_TARGETS before: ${LLVM_EXEGESIS_TARGETS}" )<br>
+<br>
 if (LLVM_TARGETS_TO_BUILD MATCHES "X86")<br>
   add_subdirectory(X86)<br>
+  message( WARNING "Appending X86" )<br>
   set(LLVM_EXEGESIS_TARGETS "${LLVM_EXEGESIS_TARGETS} X86" PARENT_SCOPE)<br>
 endif()<br>
 if (LLVM_TARGETS_TO_BUILD MATCHES "AArch64")<br>
   add_subdirectory(AArch64)<br>
+  message( WARNING "Appending AArch64" )<br>
   set(LLVM_EXEGESIS_TARGETS "${LLVM_EXEGESIS_TARGETS} AArch64" PARENT_SCOPE)<br>
 endif()<br>
 <br>
+message( WARNING "LLVM_EXEGESIS_TARGETS after: ${LLVM_EXEGESIS_TARGETS}" )<br>
+<br>
 add_library(LLVMExegesis<br>
   STATIC<br>
   Analysis.cpp<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">---<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">This results in this behavior (extraneous newlines removed for readability):<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">CMake Warning at tools/llvm-exegesis/CMakeLists.txt:11 (message):<br>
  LLVM_EXEGESIS_TARGETS at parent scope before:<br>
CMake Warning at tools/llvm-exegesis/lib/CMakeLists.txt:1 (message):<br>
  LLVM_EXEGESIS_TARGETS before:<br>
CMake Warning at tools/llvm-exegesis/lib/CMakeLists.txt:5 (message):<br>
  Appending X86<br>
CMake Warning at tools/llvm-exegesis/lib/CMakeLists.txt:10 (message):<br>
  Appending AArch64<br>
CMake Warning at tools/llvm-exegesis/lib/CMakeLists.txt:14 (message):<br>
  LLVM_EXEGESIS_TARGETS after:<br>
CMake Error at tools/llvm-exegesis/CMakeLists.txt:13 (message):<br>
  LLVM_EXEGESIS_TARGETS at parent scope after: AArch64<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">So reading the LLVM_EXEGESIS_TARGETS variable in lib/CMakeLists.txt doesn't work properly, and the second set(... PARENT_SCOPE) just sees the empty LLVM_EXEGESIS_TARGETS value, overwriting the previously appended X86 value.<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">Any ideas on how to fix this? I'm doing this locally but it seems sloppy:<o:p></o:p></p>
</div>
<div>
<p class="MsoNormal"><o:p> </o:p></p>
</div>
<div>
<p class="MsoNormal">diff --git a/tools/llvm-exegesis/lib/CMakeLists.txt b/tools/llvm-exegesis/lib/CMakeLists.txt<br>
index 175c2adf9de..194304adf98 100644<br>
--- a/tools/llvm-exegesis/lib/CMakeLists.txt<br>
+++ b/tools/llvm-exegesis/lib/CMakeLists.txt<br>
@@ -1,12 +1,16 @@<br>
+set(TARGETS_TO_APPEND "")<br>
+<br>
 if (LLVM_TARGETS_TO_BUILD MATCHES "X86")<br>
   add_subdirectory(X86)<br>
-  set(LLVM_EXEGESIS_TARGETS "${LLVM_EXEGESIS_TARGETS} X86" PARENT_SCOPE)<br>
+  set(TARGETS_TO_APPEND "${TARGETS_TO_APPEND} X86")<br>
 endif()<br>
 if (LLVM_TARGETS_TO_BUILD MATCHES "AArch64")<br>
   add_subdirectory(AArch64)<br>
-  set(LLVM_EXEGESIS_TARGETS "${LLVM_EXEGESIS_TARGETS} AArch64" PARENT_SCOPE)<br>
+  set(TARGETS_TO_APPEND "${TARGETS_TO_APPEND} AArch64")<br>
 endif()<br>
 <br>
+set(LLVM_EXEGESIS_TARGETS "${LLVM_EXEGESIS_TARGETS} ${TARGETS_TO_APPEND}" PARENT_SCOPE)<br>
+<br>
 add_library(LLVMExegesis<br>
   STATIC<br>
   Analysis.cpp<o:p></o:p></p>
</div>
</div>
</div>
</div>
</body>
</html>