FailedChanges

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

Summary

  1. [ARM][BFloat] Implement bf16 get/set_lane without casts to i16 vectors (details)
  2. [llvm] Fix typo in test case comment (NFC) (details)
  3. [mlir] Fix a doc link in LinalgBase.td (details)
  4. [flang] Fix bug checking SAVE attribute (details)
  5. [X86] Calculate the needed size of the feature arrays in _cpu_indicator_init and getHostCPUName using the size of the feature enum. (details)
  6. [X86] Remove encoding value from the X86_FEATURE and X86_FEATURE_COMPAT macro. NFCI (details)
  7. Revert "[X86] Calculate the needed size of the feature arrays in _cpu_indicator_init and getHostCPUName using the size of the feature enum." (details)
  8. [SVE] Remove calls to VectorType::getNumElements from WebASM (details)
  9. Revert "[X86][SSE] MatchVectorAllZeroTest - handle OR vector reductions" (details)
Commit 3a4feb1d53df68bed7748a0625b593ba4b5dae32 by mikhail.maltsev
[ARM][BFloat] Implement bf16 get/set_lane without casts to i16 vectors

Currently, in order to extract an element from a bf16 vector, we cast
the vector to an i16 vector, perform the extraction, and cast the result to
bfloat. This behavior was copied from the old fp16 implementation.

The goal of this patch is to achieve optimal code generation for lane
copying intrinsics in a subsequent patch (LLVM fails to fold certain
combinations of bitcast, insertelement, extractelement and
shufflevector instructions leading to the generation of suboptimal code).

Differential Revision: https://reviews.llvm.org/D82206
The file was modifiedclang/include/clang/Basic/arm_neon.td
The file was modifiedclang/lib/CodeGen/CGBuiltin.cpp
The file was modifiedclang/test/CodeGen/arm-bf16-getset-intrinsics.c
Commit 30140cd24920a2c5469f503df7b39532c7af292d by tobias.gysi
[llvm] Fix typo in test case comment (NFC)
The file was modifiedllvm/test/Transforms/Inline/always-inline.ll
Commit 809bcf4c87d01f74586a432594ac0e9c813182b7 by hanchung
[mlir] Fix a doc link in LinalgBase.td

Differential Revision: https://reviews.llvm.org/D82303
The file was modifiedmlir/include/mlir/Dialect/Linalg/IR/LinalgBase.td
Commit 663860f63e73518fc09e55a4a68b03f8027eafc8 by tkeith
[flang] Fix bug checking SAVE attribute

Treat function result like dummy argument: a SAVE statement without an
entity-list does not make it saved.

Differential Revision: https://reviews.llvm.org/D82309
The file was modifiedflang/lib/Evaluate/tools.cpp
The file was modifiedflang/test/Semantics/resolve45.f90
Commit 5ac144fe64c6911b5b6011cac1d61e0f82562e7f by craig.topper
[X86] Calculate the needed size of the feature arrays in _cpu_indicator_init and getHostCPUName using the size of the feature enum.

Move 0 initialization up to the caller so we don't need to know
the size.
The file was modifiedcompiler-rt/lib/builtins/cpu_model.c
The file was modifiedllvm/include/llvm/Support/X86TargetParser.h
The file was modifiedllvm/lib/Support/Host.cpp
Commit 0dfc8e1837e3e3ac06ab8c08fdc08b15e0ae1c2d by craig.topper
[X86] Remove encoding value from the X86_FEATURE and X86_FEATURE_COMPAT macro. NFCI

This was orignally done so we could separate the compatibility
values and the llvm internal only features into a separate entries
in the feature array. This was needed when we explicitly had to
convert the feature into the proper 32-bit chunk at every reference
and we didn't want things moving around.

Now everything is in an array and we have helper funtions or macros
to convert encoding to index. So we renumbering is no longer an
issue.
The file was modifiedllvm/include/llvm/Support/X86TargetParser.h
The file was modifiedllvm/include/llvm/Support/X86TargetParser.def
The file was modifiedclang/lib/Basic/Targets/X86.cpp
The file was modifiedclang/lib/CodeGen/CGBuiltin.cpp
Commit bebea4221d3d1db9adaf7c921c109b6c2af26b74 by craig.topper
Revert "[X86] Calculate the needed size of the feature arrays in _cpu_indicator_init and getHostCPUName using the size of the feature enum."

Seems to breaking build.

This reverts commit 5ac144fe64c6911b5b6011cac1d61e0f82562e7f.
The file was modifiedllvm/lib/Support/Host.cpp
The file was modifiedcompiler-rt/lib/builtins/cpu_model.c
The file was modifiedllvm/include/llvm/Support/X86TargetParser.h
Commit 5e2c736395cc44a00c8c2cc9fc916450bf780fbe by ctetreau
[SVE] Remove calls to VectorType::getNumElements from WebASM

Summary:
The getNumElements in base VectorType is being deprecated.

See: http://lists.llvm.org/pipermail/llvm-dev/2020-March/139811.html

Reviewers: efriedma, tlively, fpetrogalli, c-rhodes, dschuff

Reviewed By: tlively, dschuff

Subscribers: dschuff, sbc100, tschuett, jgravelle-google, hiraditya, aheejin, rkruppe, psnobl, llvm-commits

Tags: #llvm

Differential Revision: https://reviews.llvm.org/D82217
The file was modifiedllvm/lib/Target/WebAssembly/WebAssemblyTargetTransformInfo.cpp
Commit 1357c065783e8d66c9db4be59aca389d1dc6c05f by hans
Revert "[X86][SSE] MatchVectorAllZeroTest - handle OR vector reductions"

This caused a Chromium test to miscompile. See discussion on the Phabricator
review.

> This patch extends MatchVectorAllZeroTest to handle OR vector reduction patterns where the result is compared against zero.
>
> Fixes PR45378
>
> Differential Revision: https://reviews.llvm.org/D81547

This reverts 057c9c7ee00b7f7696065a3fc26a3df5ce3ebe96
The file was modifiedllvm/lib/Target/X86/X86ISelLowering.cpp
The file was modifiedllvm/test/CodeGen/X86/vector-reduce-or-cmp.ll
The file was modifiedllvm/test/CodeGen/X86/pr45378.ll