[all-commits] [llvm/llvm-project] fafa48: [AIX][clang][driver] Check the command string to t...

diggerlin via All-commits all-commits at lists.llvm.org
Tue Aug 30 07:39:20 PDT 2022


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: fafa48e7b51899f0fda80b0962679d57a1f58169
      https://github.com/llvm/llvm-project/commit/fafa48e7b51899f0fda80b0962679d57a1f58169
  Author: zhijian <zhijian at ca.ibm.com>
  Date:   2022-08-30 (Tue, 30 Aug 2022)

  Changed paths:
    M clang/docs/ReleaseNotes.rst
    M clang/include/clang/Driver/Job.h
    M clang/lib/Driver/Job.cpp
    M clang/lib/Driver/ToolChains/AIX.cpp
    M clang/test/Driver/aix-ld.c

  Log Message:
  -----------
  [AIX][clang][driver] Check the command string to the linker for exportlist opts
Summary:
Some of code in the patch are contributed by David Tenty.

1.  We currently only check driver Wl options and don't check for the plain -b, -Xlinker or other options which get passed through to the linker when we decide whether to run llvm-nm --export-symbols, so we may run it in situations where we wouldn't if the user had used the equivalent -Wl, prefixed options. If we run the export list utility when the user has specified an export list, we could export more symbols than they intended.
2.  Add a new functionality to allow redirecting the stdin, stdout, stderr of individual Jobs, if redirects are set for the Job use them, otherwise fall back to the global Compilation redirects if any.

Reviewers: David Tenty, Fangrui Song, Steven Wan
Differential Revision: https://reviews.llvm.org/D119147




More information about the All-commits mailing list