SuccessChanges

Changes from Git (git http://labmaster3.local/git/llvm-project.git)

Summary

  1. [CodeGen] Make logic of CCState::resultsCompatible clearer (details)
  2. [llvm-nm/objdump/size] Add tests for dumping symbol tables with invalid sh_size. (details)
  3. Fix error handling after [<index>] in 'frame variable' (details)
  4. Revert "[CodeGen] Make logic of CCState::resultsCompatible clearer" (details)
  5. Quiet some -Wdocumentation warnings. (details)
  6. [docs] Regenerate DiagnosticsReference.rst (details)
  7. [clang] Fix a crash on invalid auto. (details)
  8. Fix misleading comments. (details)
Commit d782d1f898eaafee49548d5332e84c3ae11ebac4 by david.spickett
[CodeGen] Make logic of CCState::resultsCompatible clearer
The file was modifiedllvm/lib/CodeGen/CallingConvLower.cpp
Commit 0054c46095eea7a10b9af8f5006160023b95046d by Xing
[llvm-nm/objdump/size] Add tests for dumping symbol tables with invalid sh_size.

This change adds tests for llvm-nm, llvm-objdump and llvm-size when dumping symbol tables with invalid sh_size (sh_size % sizeof(Elf_Sym) != 0).

Reviewed By: jhenderson, MaskRay

Differential Revision: https://reviews.llvm.org/D77864
The file was addedllvm/test/tools/llvm-nm/invalid-symbol-table-size.test
The file was addedllvm/test/tools/llvm-size/invalid-symbol-table-size.test
The file was modifiedllvm/tools/llvm-size/llvm-size.cpp
The file was addedllvm/test/tools/llvm-objdump/invalid-symbol-table-size.test
The file was modifiedllvm/tools/llvm-nm/llvm-nm.cpp
Commit cf5ed6dc59ec8e416cca91c8145b1f09d891f130 by pavel
Fix error handling after [<index>] in 'frame variable'

Summary:
This fixes a bug where

frame var a[0]+5

returns the value a[0] without any warning because the current logic simply ignores everything after ']' as long as there is no '.', '-' or '[' in the rest of the string.

The fix simplifies the termination condition of the expression path parsing loop to check if have a non-empty remaining string to parse. Previously, the condition checked if a separator was found. That condition coincided with the remaining string-to-parse condition except for the buggy indexed case where non-empty string was left ("+5" in the example above), but the separator index was 'npos'.

Reviewed By: teemperor, labath

Differential Revision: https://reviews.llvm.org/D79404
The file was modifiedlldb/source/Target/StackFrame.cpp
The file was modifiedlldb/test/API/functionalities/var_path/TestVarPath.py
Commit e1022cb5d4065aee8f5b747aac58e2cbf58c45a4 by david.spickett
Revert "[CodeGen] Make logic of CCState::resultsCompatible clearer"

This reverts commit d782d1f898eaafee49548d5332e84c3ae11ebac4
which caused test CodeGen/X86/sibcall.ll to fail.
The file was modifiedllvm/lib/CodeGen/CallingConvLower.cpp
Commit d5ea89f8914fcf7e5f637202c6972cb14617d93d by benny.kra
Quiet some -Wdocumentation warnings.
The file was modifiedllvm/include/llvm/Support/DataExtractor.h
The file was modifiedclang/include/clang/AST/PrettyPrinter.h
The file was modifiedllvm/lib/Transforms/Scalar/LoopDataPrefetch.cpp
Commit fb4574df52fd7ffed206569e74161008e95b1922 by richard.sandiford
[docs] Regenerate DiagnosticsReference.rst

It looks like it has been a while since the checked-in version of
DiagnosticsReference.rst was regenerated.  I realise there probably
isn't any expectation that the checked-in version is kept up-to-date,
but now that the project is on github and the rst can be viewed directly
from the repo's web interface, it seemed worth having something a bit
more recent.

Differential Revision: https://reviews.llvm.org/D79236
The file was modifiedclang/docs/DiagnosticsReference.rst
Commit c6e1fd70fb24d52daff4ec3fca1a89adfe7c2a0b by hokein.wu
[clang] Fix a crash on invalid auto.

Summary:
The crash is triggered on accessing a null InitExpr.

For group declaration, e.g. `auto c = a, &d = {a};`, what's happening:

1. each VarDecl is built separately during the parsing stage.
2. perform the semantic analysis (Sema::BuildDeclaratorGroup) to check
whether the type of the two VarDecl is the same, if not mark it as invalid.

in step 1, VarDecl c and d are built, both of them are valid (after D77395),
but d is without the InitExpr attached (under -fno-recovery-ast), crash
happens in step 2 when accessing the source range of d's InitExpr.

Reviewers: sammccall

Subscribers: cfe-commits

Tags: #clang

Differential Revision: https://reviews.llvm.org/D79473
The file was addedclang/test/SemaCXX/auto-invalid-init-crash.cpp
The file was modifiedclang/lib/Sema/SemaDecl.cpp
Commit be65bb12ca6590abf581dab3a0ffc4e03ab67263 by jay.foad
Fix misleading comments.
The file was modifiedllvm/include/llvm/CodeGen/Passes.h