<html><head></head><body><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 <rafael.espindola@gmail.com><br> <b><span style="font-weight: bold;">To:</span></b> reviews+D29150+public+4e541d09c9d4e079@reviews.llvm.org; mikdm@yahoo.com; dccitaliano@gmail.com; bigcheesegs@gmail.com; ruiu@google.com <br><b><span style="font-weight: bold;">Cc:</span></b> llvm-commits@lists.llvm.org; adhemerval.zanella@linaro.org<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></body></html>