SuccessChanges

Summary

  1. [llvm-objcopy] Add support for response files in llvm-strip and (details)
  2. [llvm-ar] Uncapitalize error messages and delete full stop (details)
  3. [llvm-objcopy] Default --output-target to --input-target when (details)
  4. [llvm-objcopy] Ignore -B --binary-architecture= (details)
  5. [Attributor][Fix] Use right type to replace expressions (details)
  6. [ScheduleDAGMILive] Fix typo in comment. (details)
  7. [Driver] Improve Clang::getDependencyFileName and its tests after (details)
Commit c45fd0cad428f531c30f0629852aab2fcebe7b47 by pozulp.llvm
[llvm-objcopy] Add support for response files in llvm-strip and
llvm-objcopy
Summary: Addresses https://bugs.llvm.org/show_bug.cgi?id=42671
Reviewers: jhenderson, espindola, alexshap, rupprecht
Reviewed By: jhenderson
Subscribers: seiya, emaste, arichardson, jakehehrlich, MaskRay,
abrachet, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D65372
llvm-svn: 371911
The file was addedllvm/test/tools/llvm-objcopy/ELF/response-file.test
The file was modifiedllvm/tools/llvm-objcopy/CopyConfig.cpp
The file was modifiedllvm/docs/CommandGuide/llvm-objcopy.rst
The file was modifiedllvm/tools/llvm-objcopy/llvm-objcopy.cpp
The file was modifiedllvm/docs/CommandGuide/llvm-strip.rst
The file was modifiedllvm/test/tools/llvm-objcopy/ELF/help-message.test
Commit 8a468031cd0b088d0f0b01df72d981885a6d4e89 by maskray
[llvm-ar] Uncapitalize error messages and delete full stop
Most GNU binutils don't append full stops in error messages. This
convention has been adopted by a bunch of LLVM binary utilities. Make
llvm-ar follow the convention as well.
Reviewed By: grimar
Differential Revision: https://reviews.llvm.org/D67558
llvm-svn: 371912
The file was modifiedllvm/test/Object/archive-format.test
The file was modifiedllvm/test/Object/mri3.test
The file was modifiedllvm/test/Object/mri5.test
The file was modifiedllvm/test/tools/llvm-ar/insert-before.test
The file was modifiedllvm/test/Object/mri2.test
The file was modifiedllvm/test/tools/llvm-ar/mri-addlib.test
The file was modifiedllvm/test/tools/llvm-ar/invalid-command-line.test
The file was modifiedllvm/test/tools/llvm-ar/move-before.test
The file was modifiedllvm/test/tools/llvm-ar/replace.test
The file was modifiedllvm/test/Object/mri4.test
The file was modifiedllvm/test/tools/llvm-ar/count.test
The file was modifiedllvm/test/tools/llvm-ar/insert-after.test
The file was modifiedllvm/test/Object/archive-extract.test
The file was modifiedllvm/test/tools/llvm-ar/move-after.test
The file was modifiedllvm/test/tools/llvm-ar/quick-append.test
The file was modifiedllvm/test/tools/llvm-ar/print.test
The file was modifiedllvm/test/tools/llvm-ar/move.test
The file was modifiedllvm/test/Object/archive-update.test
The file was modifiedllvm/test/tools/llvm-ar/read-only-archive.test
The file was modifiedllvm/tools/llvm-ar/llvm-ar.cpp
Commit ba53030dd0938902dd858f7eac45732295e74120 by maskray
[llvm-objcopy] Default --output-target to --input-target when
unspecified
Fixes PR42171.
In GNU objcopy, if -O (--output-target) is not specified, the value is
copied from -I (--input-target).
``` objcopy -I binary -B i386:x86-64 a.txt b       # b is copied from
a.txt llvm-objcopy -I binary -B i386:x86-64 a.txt b  # b is an x86-64
object file
```
This patch changes our behavior to match GNU. With this change, we can
delete code related to -B handling (D67215).
Reviewed By: jakehehrlich
Differential Revision: https://reviews.llvm.org/D67144
llvm-svn: 371913
The file was modifiedllvm/test/tools/llvm-objcopy/ELF/binary-input-error.test
The file was removedllvm/test/tools/llvm-objcopy/ELF/binary-input-arch.test
The file was modifiedllvm/test/tools/llvm-objcopy/ELF/binary-input.test
The file was modifiedllvm/tools/llvm-objcopy/CopyConfig.cpp
The file was modifiedllvm/test/tools/llvm-objcopy/ELF/new-symbol-visibility.test
The file was modifiedllvm/test/tools/llvm-objcopy/ELF/binary-input-and-output.test
The file was addedllvm/test/tools/llvm-objcopy/ELF/binary-output-target.test
Commit 2f519d7072bf48a81985fadc2dc145296d612223 by maskray
[llvm-objcopy] Ignore -B --binary-architecture=
GNU objcopy documents that -B is only useful with architecture-less
input (i.e. "binary" or "ihex"). After D67144, -O defaults to -I, and
-B is essentially a NOP.
* If -O is binary/ihex, GNU objcopy ignores -B.
* If -O is elf*, -B provides the e_machine field in GNU objcopy.
So to convert a blob to an ELF, `-I binary -B i386:x86-64 -O
elf64-x86-64` has to be specified.
`-I binary -B i386:x86-64 -O elf64-x86-64` creates an ELF with its
e_machine field set to EM_NONE in GNU objcopy, but a regular x86_64 ELF
in elftoolchain elfcopy. Follow the elftoolchain approach (ignoring -B)
to simplify code. Users that expect their command line portable should
specify -B.
Reviewed By: jhenderson
Differential Revision: https://reviews.llvm.org/D67215
llvm-svn: 371914
The file was modifiedllvm/test/tools/llvm-objcopy/ELF/binary-input-error.test
The file was modifiedllvm/tools/llvm-objcopy/ELF/ELFObjcopy.cpp
The file was modifiedllvm/tools/llvm-objcopy/ELF/Object.h
The file was modifiedllvm/tools/llvm-objcopy/CopyConfig.cpp
The file was modifiedllvm/tools/llvm-objcopy/CopyConfig.h
The file was modifiedllvm/docs/CommandGuide/llvm-objcopy.rst
The file was modifiedllvm/tools/llvm-objcopy/ELF/Object.cpp
The file was modifiedllvm/tools/llvm-objcopy/ObjcopyOpts.td
Commit e7c6f97039a4c9acee583af9d0e7aac94762d8ac by jdoerfert
[Attributor][Fix] Use right type to replace expressions
Summary: This should be obsolete once the functionality in D66967 is
integrated.
Reviewers: uenoku, sstefan1
Subscribers: hiraditya, bollu, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D67231
llvm-svn: 371915
The file was modifiedllvm/test/Transforms/FunctionAttrs/arg_returned.ll
The file was modifiedllvm/lib/Transforms/IPO/Attributor.cpp
Commit 4b191770f49fce1839fc3b3cabdea4c7eab60be1 by mingjie.xing
[ScheduleDAGMILive] Fix typo in comment.
Differential Revision: https://reviews.llvm.org/D67478
llvm-svn: 371916
The file was modifiedllvm/lib/CodeGen/MachineScheduler.cpp
Commit 6fe3d36768bc9323906a4a367371f98e0558a0a1 by maskray
[Driver] Improve Clang::getDependencyFileName and its tests after
rC371853
The test file name metadata-with-dots.c is confusing because -MD and
-MMD have nothing to do with metadata.
llvm-svn: 371917
The file was removedclang/test/Driver/m_and_mm.c
The file was removedclang/test/Driver/metadata-with-dots.c
The file was modifiedclang/lib/Driver/ToolChains/Clang.cpp
The file was addedclang/test/Driver/m-and-mm.c