FailedChanges

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

Summary

  1. [ARM] Always enable UseAA in the arm backend (details)
  2. Fix PR40644: miscompile indexed FP constant store (details)
  3. [Clang FE]  Recognize -mnop-mcount CL option (SystemZ only). (details)
  4. [Docs] Add LangRef documentation for freeze instruction (details)
  5. [MachineScheduler] Enable AA in PostRA Machine scheduler (details)
  6. [AtomicExpandPass] Silence static analyzer warnings about operator (details)
  7. [lldb] Fix readline/libedit compat patch for py2 (details)
  8. [InstCombine] add tests for shift-logic-shift; NFC (details)
  9. Add missing GVN =operator. NFCI. (details)
  10. [GVN] Fix uninitialized variable warnings. NFCI. (details)
  11. [JumpThreading] Factor out common code to update the SSA form (NFC) (details)
  12. [NFC][ObjC][ARC] Add tests for OptimizeRetainRVCall (details)
  13. [ObjC][ARC] Ignore lifetime markers between *ReturnValue calls (details)
  14. [OPENMP][DOCS]Fix coloring of the implemented features status, NFC. (details)
Commit cf581d7977c5c80e9f6cb6e304d7eb3d0792f360 by david.green
[ARM] Always enable UseAA in the arm backend
This feature controls whether AA is used into the backend, and was
previously turned on for certain subtargets to help create less
constrained scheduling graphs. This patch turns it on for all
subtargets, so that they can all make use of the extra information to
produce better code.
Differential Revision: https://reviews.llvm.org/D69796
The file was modifiedllvm/lib/Target/ARM/ARM.td
The file was modifiedllvm/lib/Target/ARM/ARMSubtarget.h
The file was modifiedllvm/test/CodeGen/ARM/thumb1_return_sequence.ll
The file was modifiedllvm/test/CodeGen/ARM/memcpy-ldm-stm.ll
The file was modifiedllvm/test/CodeGen/ARM/useaa.ll
The file was modifiedllvm/test/CodeGen/ARM/va_arg.ll
Commit 646896a442249380f74ff404e6dd26687f3dc6d9 by thomasp
Fix PR40644: miscompile indexed FP constant store
Summary: Functions replaceStoreOfFPConstant() and OptimizeFloatStore()
both replace store of float by a store of an integer unconditionally.
However this generates wrong code when the store that is replaced is an
indexed or truncating store. This commit solves this issue by adding an
early return in these functions when the store being considered is not a
normal store.
Bug was only observed on out of tree targets, hence the lack of testcase
in this commit.
Reviewers: efriedma
Subscribers: hiraditya, arphaman, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D68420
The file was modifiedllvm/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp
The file was modifiedllvm/lib/CodeGen/SelectionDAG/DAGCombiner.cpp
Commit 93767143147b7d765c6ce8123a4226d449228649 by paulsson
[Clang FE]  Recognize -mnop-mcount CL option (SystemZ only).
Recognize -mnop-mcount from the command line and add a function
attribute
"mnop-mcount"="true" when passed.
When this option is used, a nop is added instead of a call to fentry.
This is used when building the Linux Kernel.
If this option is passed for any other target than SystemZ, an error is
generated.
Review: Ulrich Weigand https://reviews.llvm.org/D67763
The file was modifiedclang/lib/CodeGen/CodeGenFunction.cpp
The file was modifiedclang/include/clang/Driver/Options.td
The file was modifiedclang/include/clang/Basic/CodeGenOptions.def
The file was modifiedclang/lib/Driver/ToolChains/Clang.cpp
The file was addedclang/test/CodeGen/mnop-mcount.c
The file was modifiedclang/lib/Frontend/CompilerInvocation.cpp
The file was modifiedclang/include/clang/Basic/DiagnosticCommonKinds.td
Commit 2d21068d9fa05df9e986020353224ec7449fbe68 by nunoplopes
[Docs] Add LangRef documentation for freeze instruction
Summary:
- Describe the new freeze instruction
- Make it explicit that branch on undef/poison is UB
Reviewers: chandlerc, majnemer, efriedma, nikic, reames, jdoerfert,
lebedev.ri, regehr
Subscribers: fhahn, bollu, lebedev.ri, delcypher, spatel, filcab,
llvm-commits, aqjune
Differential Revision: https://reviews.llvm.org/D29121
The file was modifiedllvm/docs/LangRef.rst
Commit f01b9aa89e8cd5d3cca0e13835302f69c1f879d2 by david.green
[MachineScheduler] Enable AA in PostRA Machine scheduler
This adds AA to Post-RA Machine Scheduling, allowing the pass more
freedom when handling memory operations.
My understanding is that this was just never done, not that it is
inherently incorrect to do so. The older PostRA List scheduler already
makes use of AA, it's just that the MI PostRA Scheduler was never taught
to use it.
Differential Revision: https://reviews.llvm.org/D69814
The file was modifiedllvm/test/CodeGen/PowerPC/extract-and-store.ll
The file was modifiedllvm/test/CodeGen/PowerPC/vec_conv_fp32_to_i64_elts.ll
The file was modifiedllvm/test/CodeGen/AArch64/merge-store-dependency.ll
The file was modifiedllvm/test/CodeGen/PowerPC/vec_conv_i16_to_fp64_elts.ll
The file was modifiedllvm/test/CodeGen/PowerPC/f128-aggregates.ll
The file was modifiedllvm/test/CodeGen/PowerPC/vec_conv_i64_to_fp32_elts.ll
The file was modifiedllvm/lib/CodeGen/MachineScheduler.cpp
Commit 9f294fc4977b361158107c389ce66d29a8f3b7ee by Dávid Bolvanský
[AtomicExpandPass] Silence static analyzer warnings about operator
priority. NFCI.
The file was modifiedllvm/lib/CodeGen/AtomicExpandPass.cpp
Commit d590498829d8c0d4f4f673569949fa3850485c9c by sguelton
[lldb] Fix readline/libedit compat patch for py2
This is a follow-up to https://reviews.llvm.org/D69793
The file was modifiedlldb/source/Plugins/ScriptInterpreter/Python/PythonReadline.cpp
Commit 3ce0c78501813e7a278573d78a0be47755ed0518 by spatel
[InstCombine] add tests for shift-logic-shift; NFC
This is based on existing CodeGen test files for x86 and AArch64. The
corresponding potential transform is shown in: rL370617
The file was addedllvm/test/Transforms/InstCombine/shift-logic.ll
Commit 1842fe6be3c341642965233a9ec7c30da277f42b by llvm-dev
Add missing GVN =operator. NFCI.
Fixes PVS Studio warning that the 'ValueTable' class implements a copy
constructor, but lacks the '=' operator.
The file was modifiedllvm/include/llvm/Transforms/Scalar/GVN.h
The file was modifiedllvm/lib/Transforms/Scalar/GVN.cpp
Commit 77debf51aba273a8d02597f90046c1f849a49add by llvm-dev
[GVN] Fix uninitialized variable warnings. NFCI.
The file was modifiedllvm/include/llvm/Transforms/Scalar/GVN.h
The file was modifiedllvm/lib/Transforms/Scalar/NewGVN.cpp
The file was modifiedllvm/lib/Transforms/Scalar/GVN.cpp
Commit 0016c1f40043e67068230208fd8f34657ad44b99 by kazu
[JumpThreading] Factor out common code to update the SSA form (NFC)
Summary: This patch factors out common code to update the SSA form in
JumpThreading.cpp -- partly for readability and partly to facilitate an
coming patch of my own.
Reviewers: wmi
Subscribers: hiraditya, jfb, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D69811
The file was modifiedllvm/include/llvm/Transforms/Scalar/JumpThreading.h
The file was modifiedllvm/lib/Transforms/Scalar/JumpThreading.cpp
Commit 68f39de042ef34ef17d93c18e80d42f2e1a970ab by francisvm
[NFC][ObjC][ARC] Add tests for OptimizeRetainRVCall
Add tests for bitcasts + zero GEPs, and pre-commit tests for lifetime
markers.
The file was modifiedllvm/test/Transforms/ObjCARC/post-inlining.ll
Commit 47d1029788b6744596bb1ab6791450eb32f2a488 by francisvm
[ObjC][ARC] Ignore lifetime markers between *ReturnValue calls
When eliminating a pair of
`llvm.objc.autoreleaseReturnValue`
followed by
`llvm.objc.retainAutoreleasedReturnValue`
we need to make sure that the instructions in between are safe to
ignore.
Other than bitcasts and useless GEPs, it's also safe to ignore lifetime
markers for both static allocas (lifetime.start/lifetime.end) and
dynamic allocas (stacksave/stackrestore).
These get added by the inliner as part of the return sequence and can
prevent the transformation from happening in practice.
Differential Revision: https://reviews.llvm.org/D69833
The file was modifiedllvm/test/Transforms/ObjCARC/post-inlining.ll
The file was modifiedllvm/lib/Transforms/ObjCARC/ObjCARCOpts.cpp
Commit 642916adc97e54810aa597512ca7012b3c8697c5 by a.bataev
[OPENMP][DOCS]Fix coloring of the implemented features status, NFC.
The file was modifiedclang/docs/OpenMPSupport.rst