SuccessChanges

Summary

  1. [macho] Allow CPUSubtype to be adjusted before writing it to a file (details)
  2. [macho] Allow CPUSubtype to contribute to architecture identification (details)
  3. Fix iOSDarwin()'s doc comment. (details)
  4. [libc++] Separate -include and path to the site config file (details)
Commit c941db5b1acdc45fc1c50762684f1014e981abc6 by daniel_l_sanders
[macho] Allow CPUSubtype to be adjusted before writing it to a file
Summary: It's not always possible for a target to provide a MachO
CPUSubtype up front as is required by the current implementation.
Sometimes you need more information like the particular CPU
implementation you are targeting.
Give MCMachObjectTargetWriter subclasses the opportunity to modify the
CPUSubtype after the MCMachObjectTargetWriter is created but before the
object starts being written. Typically this would be done in response to
instructions from a TargetStreamer.
Reviewers: pete
Subscribers: llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D70344
The file was modifiedllvm/include/llvm/MC/MCMachObjectWriter.h
Commit 17919017878e9abda5bb33a38199c593b151dcd7 by daniel_l_sanders
[macho] Allow CPUSubtype to contribute to architecture identification
Summary: Sometimes the CPUSubtype determines the Triple::ArchType that
must be used. Add the subtype to the API's to allow targets that need
this to correctly identify the contents of the binary.
Reviewers: pete
Subscribers: hiraditya, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D70345
The file was modifiedllvm/include/llvm/Object/MachO.h
The file was modifiedllvm/lib/Object/MachOObjectFile.cpp
Commit 9b36f2cfd98d56531b95c1f62976c01b65a0c18b by cyndy_ishida
Fix iOSDarwin()'s doc comment.
Summary: I saw the doc comment using the Option + Click in Xcode and I
was confused by the fact that tvOS wasn't listed. The method definition
also doesn't make it clear that the check for tvOS is taking place as
the tvOS check is hidden in the isiOS() check.
Reviewers: rjmccall
Reviewed By: rjmccall
Subscribers: dexonsmith, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D70364
This patch is from Varun Gandhi <varun_gandhi@apple.com>
The file was modifiedllvm/include/llvm/ADT/Triple.h
Commit 991831268285b955dcb459a1ea2a5f4507a33729 by Louis Dionne
[libc++] Separate -include and path to the site config file
This apparently breaks weird use cases where the build directory is on a
separate drive. Someone reported that failure to me privately.
I can't remember of a reason for collating the two arguments in the
first place, so I don't think this should break anything.
The file was modifiedlibcxx/CMakeLists.txt