[all-commits] [llvm/llvm-project] ab9741: [llvm-nm] Don't report "no symbols" error for file...

Sam Clegg via All-commits all-commits at lists.llvm.org
Fri Jan 17 09:31:24 PST 2020


  Branch: refs/heads/master
  Home:   https://github.com/llvm/llvm-project
  Commit: ab974161ba699534f3e30b1f4b036eec9c33053c
      https://github.com/llvm/llvm-project/commit/ab974161ba699534f3e30b1f4b036eec9c33053c
  Author: Sam Clegg <sbc at chromium.org>
  Date:   2020-01-17 (Fri, 17 Jan 2020)

  Changed paths:
    M llvm/test/ThinLTO/X86/empty-module.ll
    M llvm/test/ThinLTO/X86/strong_non_prevailing.ll
    A llvm/test/tools/llvm-nm/X86/nm-no-symbols-local-only.yaml
    M llvm/test/tools/llvm-nm/X86/nm-no-symbols.test
    M llvm/tools/llvm-nm/llvm-nm.cpp

  Log Message:
  -----------
  [llvm-nm] Don't report "no symbols" error for files that contain symbols

Previously we were reporting this error if we were list no symbols
which is not the same thing as the file containing no symbols.

Also, always report the filename when printing errors.

This matches the GNU nm behaviour.

This a followup to https://reviews.llvm.org/D52810

Differential Revision: https://reviews.llvm.org/D72658




More information about the All-commits mailing list