[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