[PATCH] D29150: Add --warn-unresolved-symbols and --error-unresolved-symbols options

[Dmitry Mikulin via llvm-commits]

%p/Inputs/unresolved-symbols.s already exists, I reused it for this test.I'll add -shared test points.

      From: Rafael Avila de Espindola <rafael.espindola at gmail.com>
 To: reviews+D29150+public+4e541d09c9d4e079 at reviews.llvm.org; mikdm at yahoo.com; dccitaliano at gmail.com; bigcheesegs at gmail.com; ruiu at google.com 
Cc: llvm-commits at lists.llvm.org; adhemerval.zanella at linaro.org
 Sent: Wednesday, January 25, 2017 3:31 PM
 Subject: Re: [PATCH] D29150: Add --warn-unresolved-symbols and --error-unresolved-symbols options
   
> +# RUN: llvm-mc -filetype=obj -triple=x86_64-pc-linux %p/Inputs/unresolved-symbols.s -o %t2.o

%p/Inputs/unresolved-symbols.s is missing from the patch.

Please add a test with -shared too to document the behavior: no warning/error.

Cheers,
Rafael


   
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170126/242369c4/attachment.html>