<html dir="ltr"><head>
<meta http-equiv="Content-Type" content="text/html; charset=iso-8859-1">
<meta name="GENERATOR" content="MSHTML 8.00.6001.19088">
<style id="owaTempEditStyle"></style><style title="owaParaStyle"><!--P {
MARGIN-TOP: 0px; MARGIN-BOTTOM: 0px
}
--></style>
</head>
<body ocsi="x">
<div style="FONT-FAMILY: Tahoma; DIRECTION: ltr; COLOR: #000000; FONT-SIZE: x-small">
<div>Yes, the Makefile have conditions for IntelJITEventListenerTest.cpp and OProfileJITEventListenerTest.cpp.</div>
<div><font face="tahoma"></font> </div>
<div><font face="tahoma">Regards,</font></div>
<div><font face="tahoma">Danil</font></div>
<div><font face="tahoma"></font> </div>
<div style="DIRECTION: ltr" id="divRpF31194">
<hr tabindex="-1">
<font color="#000000" size="2" face="Tahoma"><b>From:</b> Chandler Carruth [chandlerc@google.com]<br>
<b>Sent:</b> Wednesday, March 28, 2012 4:31 PM<br>
<b>To:</b> Danil Malyshev<br>
<b>Cc:</b> llvm-commits@cs.uiuc.edu<br>
<b>Subject:</b> Re: [llvm-commits] Fix missed files in JIT unittests Makefile<br>
</font><br>
</div>
<div></div>
<div>This seems fine. I guess you have to list the source files explicitly because of the conditional addition of one file?<br>
<br>
<div class="gmail_quote">On Wed, Mar 28, 2012 at 1:13 PM, Danil Malyshev <span dir="ltr">
<<a href="mailto:dmalyshev@accesssoftek.com">dmalyshev@accesssoftek.com</a>></span> wrote:<br>
<blockquote style="BORDER-LEFT: #ccc 1px solid; MARGIN: 0px 0px 0px 0.8ex; PADDING-LEFT: 1ex" class="gmail_quote">
<div>
<div style="FONT-FAMILY: Tahoma; DIRECTION: ltr; FONT-SIZE: x-small">
<div></div>
<div dir="ltr"><font color="#000000" face="Tahoma">Hi everyone,</font></div>
<div dir="ltr"><font face="tahoma"></font> </div>
<div dir="ltr"><font face="tahoma">The unittests/ExecutionEngine/JIT/Makefile has missed sourse files:
</font><font face="tahoma">JITMemoryManagerTest.cpp, </font><font face="tahoma">JITTest.cpp and
</font><font face="tahoma">MultiJITTest.cpp</font></div>
<div dir="ltr"><font face="tahoma"><font face="tahoma">S</font>o tests from these files executed only if LLVM was builded by cmake.</font></div>
<div dir="ltr"><font face="tahoma">Attached the patch fixed it.</font></div>
<div dir="ltr"><font face="tahoma"></font> </div>
<div dir="ltr"><font face="tahoma"></font> </div>
<div dir="ltr"><font face="tahoma">Regards,</font></div>
<div dir="ltr"><font face="tahoma">Danil</font></div>
</div>
</div>
<br>
_______________________________________________<br>
llvm-commits mailing list<br>
<a href="mailto:llvm-commits@cs.uiuc.edu">llvm-commits@cs.uiuc.edu</a><br>
<a href="http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits" target="_blank">http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits</a><br>
<br>
</blockquote>
</div>
<br>
</div>
</div>
</body>
</html>