UnstableChanges

Summary

  1. [Object] object::ELFObjectFile::symbol_begin(): skip symbol index 0 For clients iterating the symbol table, none expects to handle index 0 (STN_UNDEF). Skip it to improve consistency with other binary formats. Clients that need STN_UNDEF (e.g. lld) can use getSectionContentsAsArray(). A test will be added in D62148. Reviewed By: mtrent Differential Revision: https://reviews.llvm.org/D62296
  2. [cmake] When getting Ninja version, don't include CMakeNinjaFindMake which doesn't play well with passing CMAKE_MAKE_PROGRAM from the commandline without a path. Fixes a bug introduced in r361280. Thanks to Mikael Holmén for reporting this!
  3. [NFC][InstCombine] Add unary FNeg tests to maximum.ll/minimum.ll
  4. [RISCV] Support assembling TLS LA pseudo instructions This patch adds the pseudo instructions la.tls.ie and la.tls.gd, used in the initial-exec and global-dynamic TLS models respectively when addressing a global. The pseudo instructions are expanded in the assembly parser.
Revision 361506 by maskray:
[Object] object::ELFObjectFile::symbol_begin(): skip symbol index 0

For clients iterating the symbol table, none expects to handle index 0
(STN_UNDEF). Skip it to improve consistency with other binary formats.
Clients that need STN_UNDEF (e.g. lld) can use
getSectionContentsAsArray(). A test will be added in D62148.

Reviewed By: mtrent

Differential Revision: https://reviews.llvm.org/D62296
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/include/llvm/Object/ELFObjectFile.htrunk/include/llvm/Object/ELFObjectFile.h
The file was modified/llvm/trunk/tools/llvm-objdump/llvm-objdump.cpptrunk/tools/llvm-objdump/llvm-objdump.cpp
Revision 361501 by dhinton:
[cmake] When getting Ninja version, don't include CMakeNinjaFindMake
which doesn't play well with passing CMAKE_MAKE_PROGRAM from the
commandline without a path.

Fixes a bug introduced in r361280.

Thanks to Mikael Holmén for reporting this!
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/cmake/config-ix.cmaketrunk/cmake/config-ix.cmake
Revision 361500 by mcinally:
[NFC][InstCombine] Add unary FNeg tests to maximum.ll/minimum.ll
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/test/Transforms/InstCombine/maximum.lltrunk/test/Transforms/InstCombine/maximum.ll
The file was modified/llvm/trunk/test/Transforms/InstCombine/minimum.lltrunk/test/Transforms/InstCombine/minimum.ll
Revision 361499 by lewis-revill:
[RISCV] Support assembling TLS LA pseudo instructions

This patch adds the pseudo instructions la.tls.ie and la.tls.gd, used in
the initial-exec and global-dynamic TLS models respectively when
addressing a global. The pseudo instructions are expanded in the
assembly parser.
Change TypePath in RepositoryPath in Workspace
The file was modified/llvm/trunk/lib/Target/RISCV/AsmParser/RISCVAsmParser.cpptrunk/lib/Target/RISCV/AsmParser/RISCVAsmParser.cpp
The file was modified/llvm/trunk/lib/Target/RISCV/RISCVInstrInfo.tdtrunk/lib/Target/RISCV/RISCVInstrInfo.td
The file was modified/llvm/trunk/test/MC/RISCV/rvi-pseudos.strunk/test/MC/RISCV/rvi-pseudos.s

Summary

  1. [analyzer][NFC] Prettify some RUN: lines in test files. This is a test commit in disguise.
  2. [Driver] Try normalized triple when looking for C++ libraries This addresses the issue introduced in r361432 where we would only try effective triple but not the normalized one as we do for other runtimes. Differential Revision: https://reviews.llvm.org/D62286
  3. Work around a Visual C++ bug. Using a static function as a template parameter gets a bogus compile-time error with Visual Studio 2017, prior to version 15.8. Our current minimum-version requirement is a particular update to VS2015, and we assume all Visual Studio 2017 versions are usable. This patch makes the code buildable with older versions of VS2017, and can be reverted after we upgrade the minimum version sometime in the future. Description of the Microsoft bug: https://developercommunity.visualstudio.com/content/problem/25334/error-code-c2971-when-specifying-a-function-as-the.html Differential Revision: https://reviews.llvm.org/D62202
Revision 361505 by szelethus:
[analyzer][NFC] Prettify some RUN: lines in test files.

This is a test commit in disguise.
Change TypePath in RepositoryPath in Workspace
The file was modified/cfe/trunk/test/Analysis/bsd-string.ctrunk/test/Analysis/bsd-string.c
The file was modified/cfe/trunk/test/Analysis/bstring.ctrunk/test/Analysis/bstring.c
The file was modified/cfe/trunk/test/Analysis/cstring-plist.ctrunk/test/Analysis/cstring-plist.c
The file was modified/cfe/trunk/test/Analysis/null-deref-ps-region.ctrunk/test/Analysis/null-deref-ps-region.c
The file was modified/cfe/trunk/test/Analysis/string.ctrunk/test/Analysis/string.c
Revision 361504 by phosek:
[Driver] Try normalized triple when looking for C++ libraries

This addresses the issue introduced in r361432 where we would only
try effective triple but not the normalized one as we do for other
runtimes.

Differential Revision: https://reviews.llvm.org/D62286
Change TypePath in RepositoryPath in Workspace
The file was modified/cfe/trunk/lib/Driver/ToolChain.cpptrunk/lib/Driver/ToolChain.cpp
Revision 361502 by probinson:
Work around a Visual C++ bug.

Using a static function as a template parameter gets a bogus compile-time
error with Visual Studio 2017, prior to version 15.8. Our current
minimum-version requirement is a particular update to VS2015, and we
assume all Visual Studio 2017 versions are usable. This patch makes the
code buildable with older versions of VS2017, and can be reverted after
we upgrade the minimum version sometime in the future.

Description of the Microsoft bug:
https://developercommunity.visualstudio.com/content/problem/25334/error-code-c2971-when-specifying-a-function-as-the.html

Differential Revision: https://reviews.llvm.org/D62202
Change TypePath in RepositoryPath in Workspace
The file was modified/cfe/trunk/lib/Tooling/Refactoring/RangeSelector.cpptrunk/lib/Tooling/Refactoring/RangeSelector.cpp