FailedChanges

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

Summary

  1. [objc_direct] fix uniquing when re-declaring a readwrite-direct property (details)
  2. llvm-diff: Perform structural comparison on GlobalVariables, if possible (details)
  3. [Clang FE, SystemZ]  Recognize -mpacked-stack CL option (details)
  4. [WebAssembly] Convert MC tests to from bitcode to asm (details)
  5. [Attributor][NFC] Add test for sle comparison in h2s. (details)
  6. [Attributor] H2S fix. (details)
  7. [LIBOPENMP][NVPTX]Fix the build error in the runtime. (details)
  8. [OPENMP50]Add parsing/sema analysis for nontemporal clause. (details)
  9. [FPEnv] Remove unnecessary rounding mode argument for constrained (details)
  10. [perf-training] Change profile file pattern string to use %4m instead of (details)
  11. Recommit "[DebugInfo] Refactored macro related generation, added a test (details)
  12. [Frontend] Fixes -Wrange-loop-analysis warnings (details)
  13. [Sema] Fixes -Wrange-loop-analysis warnings (details)
  14. [Driver] Avoid copies in range-based for loops (details)
  15. [IR] Use a reference in a range-based for (details)
  16. Fix buildbot failures after removing REQUIRES-ANY (details)
  17. AMDGPU/SILoadStoreOptimillzer: Refactor CombineInfo struct (details)
  18. [AIX] Avoid unset csect assert for functions defined after their use in (details)
  19. [LoopFusion] Move instructions from FC0.Latch to FC1.Latch. (details)
  20. [driver][darwin] Use explicit -mlinker-version in the -platform_version (details)
  21. [FPEnv][LegalizeTypes][LegalizeDAG][AArch64] Few fixes/improvements for (details)
  22. [analysis] Discard type qualifiers when casting values retrieved from (details)
  23. Revert "[ MC ] Match labels to existing fragments even when switching (details)
  24. [WebAssembly] Implement SIMD {i8x16,i16x8}.avgr_u instructions (details)
  25. [AMDGPU] Fixed cost model for packed 16 bit ops (details)
  26. [FPEnv][LegalizeTypes] Make ScalarizeVecOp_STRICT_FP_ROUND do its own (details)
Commit a4e1819c16836dba928e646024a2406bb2eb3f94 by Alex Lorenz
[objc_direct] fix uniquing when re-declaring a readwrite-direct property
ObjCMethodDecl::getCanonicalDecl() for re-declared readwrite properties,
only looks in the ObjCInterface for the declaration of the setter
method, which it won't find.
When the method is a property accessor, we must look in extensions for a
possible redeclaration.
Radar-Id: rdar://problem/57991337
Differential Revision: https://reviews.llvm.org/D71588
The file was modifiedclang/lib/AST/DeclObjC.cpp
The file was modifiedclang/test/CodeGenObjC/direct-method.m
Commit 79b4c897b8ea5d28af1a2455c7409fc0df803079 by d.c.ddcc
llvm-diff: Perform structural comparison on GlobalVariables, if possible
Summary: Names of GlobalVariables may not be preserved depending on
compilation options, so prefer a structural diff
Subscribers: llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D71582
The file was modifiedllvm/tools/llvm-diff/DifferenceEngine.cpp
Commit 599d1cc07a51e9a556afa2a995930f7ffe0e42cd by paulsson
[Clang FE, SystemZ]  Recognize -mpacked-stack CL option
Recognize -mpacked-stack from the command line and add a function
attribute
"mpacked-stack" when passed. This is needed for 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/D71441
The file was modifiedclang/lib/Driver/ToolChains/Clang.cpp
The file was modifiedclang/include/clang/Driver/Options.td
The file was modifiedclang/lib/CodeGen/CodeGenFunction.cpp
The file was addedclang/test/CodeGen/packed-stack.c
The file was modifiedclang/include/clang/Basic/CodeGenOptions.def
The file was modifiedclang/lib/Frontend/CompilerInvocation.cpp
Commit 8aee6b414a8918993e9ae4cdabe67322ec0f6e63 by sbc
[WebAssembly] Convert MC tests to from bitcode to asm
Now that our `.s` format is stable(ish) and useable we should really
convert all our MC and lld tests over to .s format to match other
targets.
This is a test PR that just converts 2 of our MC tests to see what it
might look like.
Differential Revision: https://reviews.llvm.org/D71506
The file was addedllvm/test/MC/WebAssembly/weak.s
The file was removedllvm/test/MC/WebAssembly/weak-alias.ll
The file was removedllvm/test/MC/WebAssembly/weak.ll
The file was addedllvm/test/MC/WebAssembly/weak-alias.s
Commit d020e67575d9806d318787eec9d71828eab8f57a by sstipanovic
[Attributor][NFC] Add test for sle comparison in h2s.
The file was modifiedllvm/test/Transforms/Attributor/heap_to_stack.ll
Commit fff8ec98139379968cc7cebb71d771cf58a39a5c by sstipanovic
[Attributor] H2S fix.
Summary: Fixing issues that were noticed in D71521
Reviewers: jdoerfert, lebedev.ri, uenoku
Subscribers:
Differential Revision: https://reviews.llvm.org/D71564
The file was modifiedllvm/test/Transforms/Attributor/heap_to_stack.ll
The file was modifiedllvm/lib/Transforms/IPO/Attributor.cpp
Commit 15d47deedd1623289ef8f56cabd44bc2592745ac by a.bataev
[LIBOPENMP][NVPTX]Fix the build error in the runtime.
The file was modifiedopenmp/libomptarget/deviceRTLs/common/src/libcall.cu
Commit b6e7084e25ad0592b8e29ceea6462952e2ad79b9 by a.bataev
[OPENMP50]Add parsing/sema analysis for nontemporal clause.
Add basic support for parsing/sema analysis of the nontemporal clause in
simd-based directives.
The file was modifiedclang/test/OpenMP/taskloop_simd_misc_messages.c
The file was modifiedclang/lib/Basic/OpenMPKinds.cpp
The file was modifiedclang/test/OpenMP/target_teams_distribute_simd_ast_print.cpp
The file was modifiedclang/test/OpenMP/master_taskloop_simd_ast_print.cpp
The file was modifiedclang/test/OpenMP/parallel_for_simd_misc_messages.c
The file was modifiedclang/lib/Serialization/ASTWriter.cpp
The file was modifiedclang/test/OpenMP/teams_distribute_parallel_for_simd_messages.cpp
The file was modifiedclang/lib/Serialization/ASTReader.cpp
The file was modifiedclang/test/OpenMP/target_simd_misc_messages.c
The file was modifiedclang/include/clang/Sema/Sema.h
The file was modifiedclang/test/OpenMP/target_parallel_for_simd_misc_messages.c
The file was modifiedclang/test/OpenMP/master_taskloop_simd_misc_messages.c
The file was modifiedclang/lib/CodeGen/CGStmtOpenMP.cpp
The file was modifiedclang/test/OpenMP/parallel_master_taskloop_simd_ast_print.cpp
The file was modifiedclang/test/OpenMP/simd_ast_print.cpp
The file was modifiedclang/test/OpenMP/target_teams_distribute_parallel_for_simd_misc_messages.c
The file was modifiedclang/test/OpenMP/target_parallel_for_simd_ast_print.cpp
The file was modifiedclang/include/clang/Basic/DiagnosticSemaKinds.td
The file was modifiedclang/lib/Sema/SemaOpenMP.cpp
The file was modifiedclang/test/OpenMP/teams_distribute_parallel_for_simd_ast_print.cpp
The file was modifiedclang/include/clang/Basic/OpenMPKinds.def
The file was modifiedclang/test/OpenMP/target_teams_distribute_simd_misc_messages.c
The file was modifiedclang/include/clang/AST/RecursiveASTVisitor.h
The file was modifiedclang/test/OpenMP/target_simd_ast_print.cpp
The file was modifiedclang/tools/libclang/CIndex.cpp
The file was modifiedclang/lib/AST/StmtProfile.cpp
The file was modifiedclang/test/OpenMP/distribute_parallel_for_simd_ast_print.cpp
The file was modifiedclang/test/OpenMP/parallel_for_simd_ast_print.cpp
The file was modifiedclang/test/OpenMP/taskloop_simd_ast_print.cpp
The file was modifiedclang/lib/AST/OpenMPClause.cpp
The file was modifiedclang/lib/Parse/ParseOpenMP.cpp
The file was modifiedclang/test/OpenMP/simd_misc_messages.c
The file was modifiedclang/test/OpenMP/teams_distribute_simd_ast_print.cpp
The file was modifiedclang/test/OpenMP/parallel_master_taskloop_simd_misc_messages.c
The file was modifiedclang/test/OpenMP/for_simd_ast_print.cpp
The file was modifiedclang/include/clang/AST/OpenMPClause.h
The file was modifiedclang/test/OpenMP/distribute_parallel_for_simd_misc_messages.c
The file was modifiedclang/test/OpenMP/for_simd_misc_messages.c
The file was modifiedclang/test/OpenMP/distribute_simd_ast_print.cpp
The file was modifiedclang/test/OpenMP/distribute_simd_misc_messages.c
The file was modifiedclang/test/OpenMP/teams_distribute_simd_messages.cpp
The file was modifiedclang/lib/Sema/TreeTransform.h
Commit 1e89188d3537d419f6bdc3276ab8848adf6752f5 by ulrich.weigand
[FPEnv] Remove unnecessary rounding mode argument for constrained
intrinsics
The following intrinsics currently carry a rounding mode metadata
argument:
    llvm.experimental.constrained.minnum
   llvm.experimental.constrained.maxnum
   llvm.experimental.constrained.ceil
   llvm.experimental.constrained.floor
   llvm.experimental.constrained.round
   llvm.experimental.constrained.trunc
This is not useful since the semantics of those intrinsics do not in any
way depend on the rounding mode. In similar cases, other constrained
intrinsics do not have the rounding mode argument. Remove it here as
well.
Reviewed By: craig.topper
Differential Revision: https://reviews.llvm.org/D71218
The file was modifiedllvm/test/CodeGen/SystemZ/vec-strict-min-01.ll
The file was modifiedllvm/test/CodeGen/X86/vector-constrained-fp-intrinsics.ll
The file was modifiedllvm/test/CodeGen/PowerPC/ppcf128-constrained-fp-intrinsics.ll
The file was modifiedllvm/test/CodeGen/SystemZ/fp-strict-round-03.ll
The file was modifiedclang/test/CodeGen/constrained-math-builtins.c
The file was modifiedllvm/test/CodeGen/SystemZ/fp-strict-round-01.ll
The file was modifiedllvm/docs/LangRef.rst
The file was modifiedllvm/include/llvm/IR/ConstrainedOps.def
The file was modifiedllvm/test/CodeGen/PowerPC/vector-constrained-fp-intrinsics.ll
The file was modifiedllvm/include/llvm/IR/Intrinsics.td
The file was modifiedllvm/test/CodeGen/SystemZ/fp-strict-round-02.ll
The file was modifiedllvm/include/llvm/IR/IRBuilder.h
The file was modifiedllvm/test/CodeGen/ARM/fp-intrinsics.ll
The file was modifiedllvm/test/CodeGen/SystemZ/vec-strict-max-01.ll
The file was modifiedllvm/test/CodeGen/SystemZ/vec-strict-round-01.ll
The file was modifiedllvm/test/CodeGen/X86/fp128-libcalls-strict.ll
The file was modifiedllvm/test/CodeGen/SystemZ/vector-constrained-fp-intrinsics.ll
The file was modifiedllvm/test/CodeGen/SystemZ/vec-strict-round-02.ll
Commit b3f789e037cbfdb1439c01a4eefc9ab9bb0d2c64 by apl
[perf-training] Change profile file pattern string to use %4m instead of
%p
Summary: With %p, each test file that we're using to generate profile
data will make its own profraw file which is around 60 MB in size. If we
have a lot of test files, that quickly uses a lot of space. Use %4m
instead to share the profraw files used to store the profile data. We
use 4 here based on the default value in
https://reviews.llvm.org/source/llvm-github/browse/master/llvm/CMakeLists.txt$604
Reviewers: beanz, phosek, xiaobai, smeenai, vsk
Reviewed By: vsk
Subscribers: vsk, cfe-commits
Tags: #clang
Differential Revision: https://reviews.llvm.org/D71585
The file was modifiedclang/utils/perf-training/lit.cfg
Commit 399273e5ebac04f85257ed37934bb27f521d0617 by SourabhSingh.Tomar
Recommit "[DebugInfo] Refactored macro related generation, added a test
case for macinfo.dwo emission."
This was reverted in caa412090666c10f854322cdc701c1cbf8ed726e, since it
was causing an assertion failure on Windows bots. This revision is
revised to fix that.
Original commit message -
[DebugInfo] Refactored macro related generation, added a test case for
macinfo.dwo emission.
Reviewers: dblaikie, aprantl, jini.susan.george
Tags: #debug-info #llvm
Differential Revision: https://reviews.llvm.org/D71008
The file was modifiedllvm/lib/CodeGen/AsmPrinter/DwarfDebug.cpp
The file was addedllvm/test/DebugInfo/X86/debug-macinfo-split-dwarf.ll
The file was modifiedllvm/lib/CodeGen/AsmPrinter/DwarfDebug.h
The file was modifiedllvm/lib/MC/MCObjectFileInfo.cpp
Commit 385339034f08212d95ade89fdbacb014d86be1e2 by koraq
[Frontend] Fixes -Wrange-loop-analysis warnings
This avoids new warnings due to D68912 adds -Wrange-loop-analysis to
-Wall.
Differential Revision: https://reviews.llvm.org/D71530
The file was modifiedclang/lib/Frontend/CompilerInvocation.cpp
Commit 3ec6128daeeaadde7dd2eca8b343a56f60a8364d by koraq
[Sema] Fixes -Wrange-loop-analysis warnings
This avoids new warnings due to D68912 adds -Wrange-loop-analysis to
-Wall.
Differential Revision: https://reviews.llvm.org/D71529
The file was modifiedclang/lib/Sema/SemaCodeComplete.cpp
The file was modifiedclang/lib/Sema/AnalysisBasedWarnings.cpp
Commit b750486c5d96320daf77a9760166f78b4a0c942e by koraq
[Driver] Avoid copies in range-based for loops
This avoids new warnings due to D68912 adds -Wrange-loop-analysis to
-Wall.
Differential Revision: https://reviews.llvm.org/D71527
The file was modifiedclang/lib/Driver/Driver.cpp
The file was modifiedclang/lib/Driver/ToolChain.cpp
Commit 1a8ff89653d2a80a013701fe927d2d32491bff59 by koraq
[IR] Use a reference in a range-based for
This avoids unneeded copies when using a range-based for loops.
This avoids new warnings due to D68912 adds -Wrange-loop-analysis to
-Wall.
Differential Revision: https://reviews.llvm.org/D70870
The file was modifiedllvm/lib/IR/Attributes.cpp
The file was modifiedllvm/lib/IR/ModuleSummaryIndex.cpp
Commit ba5a00167bf30df5d544fdbe9fd28ce1a8341b89 by nemanja.i.ibm
Fix buildbot failures after removing REQUIRES-ANY
It would appear that the removal of this lit feature was incomplete and
there is a test case that still tests for this. This patch removes the
remaining tests to bring the bots back to green. I would encourage the
author to do a post-commit review on this in case there is a more
desirable fix.
The file was modifiedllvm/utils/lit/tests/shtest-format.py
Commit c3bc805f4f99d7a75d0d45ce97ade6ade554e505 by tstellar
AMDGPU/SILoadStoreOptimillzer: Refactor CombineInfo struct
Summary: Modify CombineInfo to only store information about a single
instruction. This is a little easier to work with and removes a lot of
duplicate initialization code.
Reviewers: arsenm, nhaehnle
Reviewed By: arsenm, nhaehnle
Subscribers: merge_guards_bot, kzhuravl, jvesely, wdng, yaxunl,
dstuttard, tpr, t-tye, hiraditya, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D71045
The file was modifiedllvm/lib/Target/AMDGPU/SILoadStoreOptimizer.cpp
Commit 84161f18ccdc8f8f5a8fc16ca16d402ddc2b9ed4 by daltenty
[AIX] Avoid unset csect assert for functions defined after their use in
TOC
Summary: If a function is defined after it appears in a TOC expression,
we may try to access an unset containing csect when returning a symbol
for the expression.
Reviewers: Xiangling_L, DiggerLin, jasonliu, hubert.reinterpretcast
Reviewed By: hubert.reinterpretcast
Subscribers: hubert.reinterpretcast, wuzish, nemanjai, hiraditya,
kbarton, jsji, llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D71125
The file was modifiedllvm/lib/Target/PowerPC/PPCAsmPrinter.cpp
The file was modifiedllvm/test/CodeGen/PowerPC/aix-xcoff-toc.ll
Commit 36bdc3dc35a509450d52a79225b1509f587236e9 by whitneyt
[LoopFusion] Move instructions from FC0.Latch to FC1.Latch.
Summary:This PR move instructions from FC0.Latch bottom up to the
beginning of FC1.Latch as long as they are proven safe.
To illustrate why this is beneficial, let's consider the following
example: Before Fusion: header1:
br header2 header2:
br header2, latch1 latch1:
br header1, preheader3 preheader3:
br header3 header3:
br header4 header4:
br header4, latch3 latch3:
br header3, exit3
After Fusion (before this PR): header1:
br header2 header2:
br header2, latch1 latch1:
br header3 header3:
br header4 header4:
br header4, latch3 latch3:
br header1, exit3
Note that preheader3 is removed during fusion before this PR. Notice
that we cannot fuse loop2 with loop4 as there exists block latch1 in
between. This PR move instructions from latch1 to beginning of latch3,
and remove block latch1. LoopFusion is now able to fuse loop nest
recursively.
After Fusion (after this PR): header1:
br header2 header2:
br header3 header3:
br header4 header4:
br header2, latch3 latch3:
br header1, exit3
Reviewer: kbarton, jdoerfert, Meinersbur, dmgreen, fhahn, hfinkel,
bmahjour, etiotto Reviewed By: kbarton, Meinersbur Subscribers:
hiraditya, llvm-commits Tag: LLVM Differential Revision:
https://reviews.llvm.org/D71165
The file was modifiedllvm/test/Transforms/LoopFusion/loop_nest.ll
The file was modifiedllvm/test/Transforms/LoopFusion/four_loops.ll
The file was modifiedllvm/lib/Transforms/Scalar/LoopFuse.cpp
The file was modifiedllvm/test/Transforms/LoopFusion/simple.ll
The file was modifiedllvm/test/Transforms/LoopFusion/guarded.ll
The file was modifiedllvm/lib/Transforms/Utils/CodeMoverUtils.cpp
The file was modifiedllvm/include/llvm/Transforms/Utils/CodeMoverUtils.h
Commit be88a20c900463c4919433109e4c17cd001da580 by Alex Lorenz
[driver][darwin] Use explicit -mlinker-version in the -platform_version
tests
The driver actually adds a default -mlinker-version, based on
HOST_LINK_VERSION cmake variable. The tests should be explicit about
which version they're using to trigger the right behavior.
The file was modifiedclang/test/Driver/darwin-ld-platform-version-watchos.c
The file was modifiedclang/test/Driver/darwin-ld-platform-version-ios.c
The file was modifiedclang/test/Driver/darwin-ld-platform-version-tvos.c
The file was modifiedclang/test/Driver/darwin-ld-platform-version-macos.c
Commit 84d8fa30f93a58a360524b93925ebfb0d36eba2e by craig.topper
[FPEnv][LegalizeTypes][LegalizeDAG][AArch64] Few fixes/improvements for
legalizing fp<->int conversion nodes.
This started with adding a test to support get code coverage on
ScalarizeVecOp_UnaryOp_StrictFP by copying an existing AArch64 test and
using constrained sitofp/uitofp intrinsics.
This found 3 separate issues:
-ScalarizeVecOp_UnaryOp_StrictFP needs to do its own replacement
because the caller can't handle replacing multiple results.
-Missing integer promotion support for sitofp/uitofp
-Chain result not always assigned in ExpandLegalINT_TO_FP.
Committing them together so I can add the test case.
The file was addedllvm/test/CodeGen/AArch64/fpconv-vector-op-scalarize-strict.ll
The file was modifiedllvm/lib/CodeGen/SelectionDAG/LegalizeDAG.cpp
The file was modifiedllvm/lib/CodeGen/SelectionDAG/LegalizeVectorTypes.cpp
The file was modifiedllvm/lib/CodeGen/SelectionDAG/LegalizeTypes.h
The file was modifiedllvm/lib/CodeGen/SelectionDAG/LegalizeIntegerTypes.cpp
Commit 6d3f43ec61a60c37963ee5f54289cf0759fb5d61 by Artem Dergachev
[analysis] Discard type qualifiers when casting values retrieved from
the Store.
This canonicalizes the representation of unknown pointer symbols, which
reduces the overall confusion in pointer cast representation.
Patch by Vince Bridgers!
Differential Revision: https://reviews.llvm.org/D70836
The file was addedclang/test/Analysis/uninit-val-const-likeness.c
The file was modifiedclang/lib/StaticAnalyzer/Core/Store.cpp
Commit f827aff8598873194bccdfaf469f2dde7e5620d1 by 31459023+hctim
Revert "[ MC ] Match labels to existing fragments even when switching
sections."
This reverts commit 4272372c571cd33edc77a8844b0a224ad7339138.
Caused an MSan buildbot failure. More information available in the patch
that introduced the bug: https://reviews.llvm.org/D71368
The file was modifiedllvm/lib/MC/MCSection.cpp
The file was modifiedllvm/include/llvm/MC/MCObjectStreamer.h
The file was modifiedllvm/lib/MC/MCObjectStreamer.cpp
The file was removedllvm/test/MC/MachO/pending-labels.s
The file was modifiedllvm/include/llvm/MC/MCSection.h
Commit f1b351e14a912dbb797b731807d81d4c9af4db0e by tlively
[WebAssembly] Implement SIMD {i8x16,i16x8}.avgr_u instructions
Summary: These instructions were added to the spec proposal in
https://github.com/WebAssembly/simd/pull/126. Their semantics are
equivalent to `(a + b + 1) / 2`. The opcode for the experimental
i32x4.dot_i16x8_s is also bumped due to a collision with the
i8x16.avgr_u opcode.
Reviewers: aheejin
Subscribers: dschuff, sbc100, jgravelle-google, hiraditya, sunfish,
llvm-commits
Tags: #llvm
Differential Revision: https://reviews.llvm.org/D71628
The file was modifiedllvm/test/CodeGen/WebAssembly/simd-arith.ll
The file was modifiedllvm/test/MC/WebAssembly/simd-encodings.s
The file was modifiedllvm/lib/Target/WebAssembly/WebAssemblyInstrSIMD.td
Commit b8ac5894a115987fcc7e871049ec31a8eba66741 by Stanislav.Mekhanoshin
[AMDGPU] Fixed cost model for packed 16 bit ops
Differential Revision: https://reviews.llvm.org/D71622
The file was modifiedllvm/test/Analysis/CostModel/AMDGPU/add-sub.ll
The file was modifiedllvm/test/Analysis/CostModel/AMDGPU/bit-ops.ll
The file was modifiedllvm/test/Analysis/CostModel/AMDGPU/shifts.ll
The file was modifiedllvm/test/Analysis/CostModel/AMDGPU/fadd.ll
The file was modifiedllvm/test/Analysis/CostModel/AMDGPU/mul.ll
The file was modifiedllvm/test/Analysis/CostModel/AMDGPU/fmul.ll
The file was modifiedllvm/lib/Target/AMDGPU/AMDGPUTargetTransformInfo.cpp
The file was modifiedllvm/test/Analysis/CostModel/AMDGPU/fsub.ll
Commit c36773c78eb52ae0c549dc286a441af0d1b8b712 by craig.topper
[FPEnv][LegalizeTypes] Make ScalarizeVecOp_STRICT_FP_ROUND do its own
replacements and return SDValue()
The caller will assert for nodes with more than 2 results unless we
return a null SDValue.
I tried to test this by copying an AArch64 test for
ScalarizeVecOp_FP_ROUND. While it did hit the assert and this commited
fixed that. It also hit a later problem that couldn't be fixed without
adding strict FP support to AArch64.
The file was modifiedllvm/lib/CodeGen/SelectionDAG/LegalizeVectorTypes.cpp