SuccessChanges

Summary

  1. [lib/Object] - Make ELFObjectFile::getSymbol() return Expected<>. (details)
  2. [yaml2obj/obj2yaml] - Make Value/Size fields of Symbol optional. (details)
  3. Revert "[LLDB] Remove AArch64/Linux xfail decorator from TestGuiBasicDebug" (details)
  4. [X86][SSE] combineReductionToHorizontal - don't rely on widenSubVector to handle illegal vector types. (details)
  5. Revert "Ensure SplitEdge to return the new block between the two given blocks" (details)
Commit 407d42002904ce541f732ce4300913ef57cff232 by grimar
[lib/Object] - Make ELFObjectFile::getSymbol() return Expected<>.

This was requested in comments for D93209:
https://reviews.llvm.org/D93209#inline-871192

D93209 fixes an issue with `ELFFile<ELFT>::getEntry`,
after what `getSymbol` starts calling `report_fatal_error` for previously
missed invalid cases.

This patch makes it return `Expected<>` and updates callers.
For few of them I had to add new `report_fatal_error` calls. But I see no
way to avoid it currently. The change would affects too many places, e.g:
`getSymbolBinding` and other methods are used from `ELFSymbolRef`
which is used in too many places across LLVM.

Differential revision: https://reviews.llvm.org/D93297
The file was modifiedllvm/tools/llvm-objdump/llvm-objdump.cpp
The file was modifiedllvm/unittests/Object/ELFObjectFileTest.cpp
The file was modifiedllvm/tools/llvm-objdump/ELFDump.cpp
The file was modifiedllvm/include/llvm/Object/ELFObjectFile.h
Commit 8c2cf89834c3496be343525b9f9c6aecc9182117 by grimar
[yaml2obj/obj2yaml] - Make Value/Size fields of Symbol optional.

When a field is optional we can use the `=<none>` syntax in macros.
This patch makes `Value`/`Size` fields of `Symbol` optional
and adds test cases for them.

Differential revision: https://reviews.llvm.org/D93010
The file was addedllvm/test/tools/yaml2obj/ELF/symbol-size.yaml
The file was modifiedllvm/lib/ObjectYAML/ELFYAML.cpp
The file was modifiedllvm/include/llvm/ObjectYAML/ELFYAML.h
The file was addedllvm/test/tools/obj2yaml/ELF/symbol.yaml
The file was addedllvm/test/tools/yaml2obj/ELF/symbol-value.yaml
The file was modifiedllvm/tools/obj2yaml/elf2yaml.cpp
The file was modifiedllvm/lib/ObjectYAML/ELFEmitter.cpp
Commit 9f80ab1213e9f28b1b86f133fa7edf9a61c6f8fd by omair.javaid
Revert "[LLDB] Remove AArch64/Linux xfail decorator from TestGuiBasicDebug"

This reverts commit 3d27a99b2ed24e1951483cf13357ec188ad44bb0.
The file was modifiedlldb/test/API/commands/gui/basicdebug/TestGuiBasicDebug.py
Commit e55f7de946b1be546b05ce7399ec4f8fc7980cf5 by llvm-dev
[X86][SSE] combineReductionToHorizontal - don't rely on widenSubVector to handle illegal vector types.

Thanks to @asbirlea for reporting the bug.
The file was modifiedllvm/test/CodeGen/X86/vector-reduce-mul.ll
The file was modifiedllvm/lib/Target/X86/X86ISelLowering.cpp
Commit c10757200d89e59a52ab8cbbc68178eeafaa3600 by whitneyt
Revert "Ensure SplitEdge to return the new block between the two given blocks"

This reverts commit cf638d793c489632bbcf0ee0fbf9d0f8c76e1f48.
The file was modifiedllvm/lib/IR/BasicBlock.cpp
The file was modifiedllvm/test/Transforms/LoopUnswitch/2011-11-18-SimpleSwitch.ll
The file was modifiedllvm/include/llvm/IR/BasicBlock.h
The file was modifiedllvm/test/CodeGen/AMDGPU/call-constexpr.ll
The file was modifiedllvm/lib/Transforms/Utils/BasicBlockUtils.cpp
The file was modifiedllvm/include/llvm/Transforms/Utils/BasicBlockUtils.h
The file was modifiedllvm/unittests/Transforms/Utils/BasicBlockUtilsTest.cpp