SuccessChanges

Summary

  1. SemaExprCXX - silence static analyzer getAs<> null dereference warnings. (details)
  2. clang-tidy - silence static analyzer getAs<> null dereference warnings. (details)
  3. JumpThreadingPass::UnfoldSelectInstr - silence static analyzer (details)
  4. Include leading attributes in DeclStmt's SourceRange (details)
  5. [llvm-objcopy] --add-symbol: fix crash if SHT_SYMTAB does not exist (details)
  6. [llvm-ar] Implement the O modifier: display member offsets inside the (details)
Commit e14851259e770a59db736964475ebd152fe1c793 by llvm-dev
SemaExprCXX - silence static analyzer getAs<> null dereference warnings.
NFCI.
The static analyzer is warning about potential null dereferences, but in
these cases we should be able to use castAs<> directly and if not assert
will fire for us.
llvm-svn: 375101
The file was modifiedclang/lib/Sema/SemaExprCXX.cpp
Commit 2ea8b58cb6182b0e0a8982849a1702406abfc99b by llvm-dev
clang-tidy - silence static analyzer getAs<> null dereference warnings.
NFCI.
The static analyzer is warning about potential null dereferences, but in
these cases we should be able to use castAs<> directly and if not assert
will fire for us.
llvm-svn: 375102
The file was modifiedclang-tools-extra/clang-tidy/modernize/UseDefaultMemberInitCheck.cpp
The file was modifiedclang-tools-extra/clang-tidy/misc/RedundantExpressionCheck.cpp
Commit 3ec83e81877da19254adcdb2e80861eec19ea928 by llvm-dev
JumpThreadingPass::UnfoldSelectInstr - silence static analyzer
dyn_cast<> null dereference warning. NFCI.
The static analyzer is warning about a potential null dereference, but
we should be able to use cast<> directly and if not assert will fire for
us.
llvm-svn: 375103
The file was modifiedllvm/lib/Transforms/Scalar/JumpThreading.cpp
Commit dc3957ec215dd17b8d293461f18696566637a6cd by sbergman
Include leading attributes in DeclStmt's SourceRange
Differential Revision: https://reviews.llvm.org/D68581
llvm-svn: 375104
The file was modifiedclang/test/AST/sourceranges.cpp
The file was modifiedclang/lib/Parse/ParseStmt.cpp
Commit 9dce25a9fa953cec1b89009226cdc463166a7ad4 by maskray
[llvm-objcopy] --add-symbol: fix crash if SHT_SYMTAB does not exist
Exposed by D69041. If SHT_SYMTAB does not exist,
ELFObjcopy.cpp:handleArgs will crash due to a null pointer dereference.
  for (const NewSymbolInfo &SI : Config.ELF->SymbolsToAdd) {
   ...
   Obj.SymbolTable->addSymbol(
Fix this by creating .symtab and .strtab on demand in
ELFBuilder<ELFT>::readSections, if --add-symbol is specified.
Reviewed By: grimar
Differential Revision: https://reviews.llvm.org/D69093
llvm-svn: 375105
The file was addedllvm/test/tools/llvm-objcopy/ELF/add-symbol-no-symtab.test
The file was modifiedllvm/tools/llvm-objcopy/ELF/Object.h
The file was modifiedllvm/tools/llvm-objcopy/ELF/ELFObjcopy.cpp
The file was modifiedllvm/tools/llvm-objcopy/ELF/Object.cpp
Commit a69cc92cb5083d9d9f33c19587db1cdcab742637 by maskray
[llvm-ar] Implement the O modifier: display member offsets inside the
archive
Since GNU ar 2.31, the 't' operation prints member offsets beside file
names if the 'O' modifier is specified. 'O' is ignored for thin
archives.
Reviewed By: gbreynoo, ruiu
Differential Revision: https://reviews.llvm.org/D69087
llvm-svn: 375106
The file was modifiedllvm/include/llvm/Object/Archive.h
The file was modifiedllvm/test/Object/archive-toc.test
The file was modifiedllvm/docs/CommandGuide/llvm-ar.rst
The file was modifiedllvm/tools/llvm-ar/llvm-ar.cpp
The file was modifiedllvm/test/Object/archive-thin-create.test