<html>
<head>
<meta http-equiv="Content-Type" content="text/html; charset=utf-8">
</head>
<body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space; color: rgb(0, 0, 0); font-size: 14px; font-family: Calibri, sans-serif;">
<div>
<div>
<div>Sending from the right account…</div>
<div>
<div id="MAC_OUTLOOK_SIGNATURE"></div>
</div>
</div>
</div>
<div><br>
</div>
<span id="OLK_SRC_BODY_SECTION">
<div style="font-family:Calibri; font-size:12pt; text-align:left; color:black; BORDER-BOTTOM: medium none; BORDER-LEFT: medium none; PADDING-BOTTOM: 0in; PADDING-LEFT: 0in; PADDING-RIGHT: 0in; BORDER-TOP: #b5c4df 1pt solid; BORDER-RIGHT: medium none; PADDING-TOP: 3pt">
<span style="font-weight:bold">From: </span>Dmitry Mikulin<br>
<span style="font-weight:bold">Reply-To: </span>Dmitry Mikulin<br>
<span style="font-weight:bold">Date: </span>Wednesday, January 25, 2017 at 4:13 PM<br>
<span style="font-weight:bold">To: </span>Rafael Avila de Espindola, "<a href="mailto:reviews+D29150+public+4e541d09c9d4e079@reviews.llvm.org">reviews+D29150+public+4e541d09c9d4e079@reviews.llvm.org</a>", "<a href="mailto:dccitaliano@gmail.com">dccitaliano@gmail.com</a>",
"<a href="mailto:bigcheesegs@gmail.com">bigcheesegs@gmail.com</a>", "<a href="mailto:ruiu@google.com">ruiu@google.com</a>", Dmitry Mikulin<br>
<span style="font-weight:bold">Cc: </span>"<a href="mailto:llvm-commits@lists.llvm.org">llvm-commits@lists.llvm.org</a>", "<a href="mailto:adhemerval.zanella@linaro.org">adhemerval.zanella@linaro.org</a>"<br>
<span style="font-weight:bold">Subject: </span>Re: [PATCH] D29150: Add --warn-unresolved-symbols and --error-unresolved-symbols options<br>
</div>
<div><br>
</div>
<div>
<div>
<div style="color:#000; background-color:#fff; font-family:HelveticaNeue, Helvetica Neue, Helvetica, Arial, Lucida Grande, sans-serif;font-size:10px">
<div id="yui_3_16_0_ym19_1_1485389438054_8996" dir="ltr"><span style="font-size: 16px;" id="yui_3_16_0_ym19_1_1485389438054_9071">%p/Inputs/unresolved-symbols.s already exists, I reused it for this test.</span><span></span></div>
<div id="yui_3_16_0_ym19_1_1485389438054_8996" dir="ltr"><span style="font-size: 16px;" id="yui_3_16_0_ym19_1_1485389438054_9094">I'll add -shared test points.</span></div>
<div class="qtdSeparateBR" id="yui_3_16_0_ym19_1_1485389438054_8997"><br>
<br>
</div>
<div class="yahoo_quoted" id="yui_3_16_0_ym19_1_1485389438054_9058" style="display: block;">
<div style="font-family: HelveticaNeue, Helvetica Neue, Helvetica, Arial, Lucida Grande, sans-serif; font-size: 10px;" id="yui_3_16_0_ym19_1_1485389438054_9057">
<div style="font-family: HelveticaNeue, Helvetica Neue, Helvetica, Arial, Lucida Grande, sans-serif; font-size: 16px;" id="yui_3_16_0_ym19_1_1485389438054_9056">
<div dir="ltr" id="yui_3_16_0_ym19_1_1485389438054_9055"><font size="2" face="Arial" id="yui_3_16_0_ym19_1_1485389438054_9054">
<hr size="1" id="yui_3_16_0_ym19_1_1485389438054_9068">
<b><span style="font-weight:bold;">From:</span></b> Rafael Avila de Espindola <<a href="mailto:rafael.espindola@gmail.com">rafael.espindola@gmail.com</a>><br>
<b><span style="font-weight: bold;">To:</span></b> <a href="mailto:reviews+D29150+public+4e541d09c9d4e079@reviews.llvm.org">
reviews+D29150+public+4e541d09c9d4e079@reviews.llvm.org</a>; <a href="mailto:mikdm@yahoo.com">
mikdm@yahoo.com</a>; <a href="mailto:dccitaliano@gmail.com">dccitaliano@gmail.com</a>;
<a href="mailto:bigcheesegs@gmail.com">bigcheesegs@gmail.com</a>; <a href="mailto:ruiu@google.com">
ruiu@google.com</a> <br>
<b><span style="font-weight: bold;">Cc:</span></b> <a href="mailto:llvm-commits@lists.llvm.org">
llvm-commits@lists.llvm.org</a>; <a href="mailto:adhemerval.zanella@linaro.org">adhemerval.zanella@linaro.org</a><br>
<b><span style="font-weight: bold;">Sent:</span></b> Wednesday, January 25, 2017 3:31 PM<br>
<b><span style="font-weight: bold;">Subject:</span></b> Re: [PATCH] D29150: Add --warn-unresolved-symbols and --error-unresolved-symbols options<br>
</font></div>
<div class="y_msg_container" id="yui_3_16_0_ym19_1_1485389438054_9059"><br>
<div class="yqt4563736186" id="yqtfd18721">> +# RUN: llvm-mc -filetype=obj -triple=x86_64-pc-linux %p/Inputs/unresolved-symbols.s -o %t2.o</div>
<br clear="none">
<br clear="none">
%p/Inputs/unresolved-symbols.s is missing from the patch.<br clear="none">
<br clear="none">
Please add a test with -shared too to document the behavior: no warning/error.<br clear="none">
<br clear="none">
Cheers,<br clear="none">
Rafael
<div class="yqt4563736186" id="yqtfd84009"><br clear="none">
</div>
<br>
<br>
</div>
</div>
</div>
</div>
</div>
</div>
</div>
</span>
</body>
</html>