FailedChanges

Summary

  1. [lldb] Look beyond the first line to find the PID in TestAppleSimulatorOSType (details)
  2. [SLPVectorizer] Pre-commit a test for D85759 (details)
  3. Update AMDGPU testcases after bebe6a6449811e877f7eba3f1798ddd1fa83e440 (details)
  4. [lldb] Enable inheriting TCC permissions in lldb-test (details)
  5. [BPI] Teach BPI about bcmp function (details)
  6. [OpenMP] split execution of a long test into smaller parts. (details)
  7. [compiler-rt] XFAIL test on iOS (details)
  8. NFC. Constify MachineVerifier::verify parameter (details)
  9. Revert "[SLPVectorizer] Pre-commit a test for D85759" (details)
  10. Reland "[SLPVectorizer] Pre-commit a test for D85759" (details)
  11. [libunwind][NFC] Fix typo in comment (details)
  12. [AIX][XCOFF] change the operand of branch instruction from symbol name to qualified symbol name for function declarations (details)
  13. [WebAssembly][NFC] Replace WASM with standard Wasm (details)
  14. [gn build] Fix CodeGenTests (details)
  15. [libunwind] Fix incorrect check for out-of-boundedness (details)
  16. [NFC][InstCombine] Add tests for PHI merging/aggregate reconstruction (PR47060) (details)
  17. [lldb] Fix the last remaining tests not inheriting TCC permissions (details)
  18. [InstSimplify] Respect CanUseUndef in more places (details)
Commit 7adf5bd18147b8acdab6e2720b4073e8c35bdf9a by Jonas Devlieghere
[lldb] Look beyond the first line to find the PID in TestAppleSimulatorOSType

The current code fails when the first stderr line doesn't match the
given regex to parse the PID. This patch changes the code to read the
first 10 lines before giving up. It also adds tracing for the simctl
commands.
The file was modifiedlldb/test/API/tools/lldb-server/TestAppleSimulatorOSType.py
Commit 94791970de109eb9a6b296825ddb0fc2a196b366 by tlively
[SLPVectorizer] Pre-commit a test for D85759

8cc911fa5b06 refactored the `getIntrinsicInstrCost` function and was
meant to be a nonfunctional change, but it accidentally changed how
costs were calculated in the SLP vectorizer, which regressed
WebAssembly codegen and resulted in a downstream bug report at
https://github.com/emscripten-core/emscripten/issues/11449.

The fix for this regression is in D85759, and this patch just
pre-commits the test from that patch to demonstrate the regressed
behavior first.
The file was addedllvm/test/Transforms/SLPVectorizer/WebAssembly/no-vectorize-rotate.ll
Commit cf9588a24a276e36ac050e9172a33d9b431f12e9 by Jessica Paquette
Update AMDGPU testcases after bebe6a6449811e877f7eba3f1798ddd1fa83e440

I didn't build AMDGPU locally so I didn't see this.

```
(logic_op (op x...), (op y...)) -> (op (logic_op x, y))
```

kicks in here.

Differential Revision: https://reviews.llvm.org/D85761
The file was modifiedllvm/test/CodeGen/AMDGPU/GlobalISel/orn2.ll
The file was modifiedllvm/test/CodeGen/AMDGPU/GlobalISel/xnor.ll
The file was modifiedllvm/test/CodeGen/AMDGPU/GlobalISel/andn2.ll
Commit 61afdf0ab43f1aec93d3d482cd3d1d95c537aefe by Jonas Devlieghere
[lldb] Enable inheriting TCC permissions in lldb-test

Like the rest of the test suite, also set the target.inherit-tcc option
to true in lldb-test.
The file was modifiedlldb/tools/lldb-test/lldb-test.cpp
Commit d68a2859ab410426edf1683d926fa2d1b58c5935 by Dávid Bolvanský
[BPI] Teach BPI about bcmp function
bcmp is similar to memcmp
The file was modifiedllvm/lib/Analysis/BranchProbabilityInfo.cpp
The file was modifiedllvm/test/Analysis/BranchProbabilityInfo/libfunc_call.ll
Commit ec5f793996f4dc86d339db88c0836e0cf4e8abea by tra
[OpenMP] split execution of a long test into smaller parts.

This test is bottlenecked by heavy regex use (~0.6s per FileCHeck run) with the
content that can't be further fragmented. Instead, the test body is moved into a
common .inc file and test execution has been split into four roughly equal
parts. This reduces wall time for the test from 14s to ~3.5s.

Differential Revision: https://reviews.llvm.org/D85695
The file was addedclang/test/OpenMP/target_map_codegen_18.inc
The file was addedclang/test/OpenMP/target_map_codegen_18c.cpp
The file was addedclang/test/OpenMP/target_map_codegen_18a.cpp
The file was addedclang/test/OpenMP/target_map_codegen_18d.cpp
The file was removedclang/test/OpenMP/target_map_codegen_18.cpp
The file was addedclang/test/OpenMP/target_map_codegen_18b.cpp
Commit bfb0b853b2da6634df4459d844be7ac4739b4e50 by julian.lettner
[compiler-rt] XFAIL test on iOS

On iOS, when we `longjmp()` out of the signal handler, a subsequent call
to `sigaltstack()` still reports that we are executing on the signal
handler stack.
Tracking rdar://66789814

Differential Revision: https://reviews.llvm.org/D85677
The file was modifiedcompiler-rt/test/asan/TestCases/Posix/unpoison-alternate-stack.cpp
Commit 39617aaed95ac00957979bc1525598c1be80e85e by Yuanfang Chen
NFC. Constify MachineVerifier::verify parameter
The file was modifiedllvm/lib/CodeGen/MachineVerifier.cpp
Commit 52b71aa8b1a019403b0ecc184744b2f8ca2f7cba by tlively
Revert "[SLPVectorizer] Pre-commit a test for D85759"

This reverts commit 94791970de109eb9a6b296825ddb0fc2a196b366.

The test is failing on multiple bots, event though it passes for me
locally. Reverting while I investigate further.
The file was removedllvm/test/Transforms/SLPVectorizer/WebAssembly/no-vectorize-rotate.ll
Commit f969734c21e89a0e26f2490347d7be750500774e by tlively
Reland "[SLPVectorizer] Pre-commit a test for D85759"

This reverts commit 52b71aa8b1a019403b0ecc184744b2f8ca2f7cba.

The problem was a missing lit.local.cfg file, which was causing the
test to be incorrectly run on bots that had not built the WebAssembly
target.
The file was addedllvm/test/Transforms/SLPVectorizer/WebAssembly/lit.local.cfg
The file was addedllvm/test/Transforms/SLPVectorizer/WebAssembly/no-vectorize-rotate.ll
Commit e8dac8b3dbe7a31af291032f1f3e95e789200590 by Louis Dionne
[libunwind][NFC] Fix typo in comment
The file was modifiedlibunwind/src/UnwindCursor.hpp
Commit e9ac1495e2082ebacfe14df2ffd124a58860449d by diggerlin
[AIX][XCOFF] change the operand of branch instruction from symbol name to qualified symbol name for function declarations

SUMMARY:

1. in the patch  , remove setting storageclass in function .getXCOFFSection and construct function of class MCSectionXCOFF
there are

XCOFF::StorageMappingClass MappingClass;
XCOFF::SymbolType Type;
XCOFF::StorageClass StorageClass;
in the MCSectionXCOFF class,
these attribute only used in the XCOFFObjectWriter, (asm path do not need the StorageClass)

we need get the value of StorageClass, Type,MappingClass before we invoke the getXCOFFSection every time.

actually , we can get the StorageClass of the MCSectionXCOFF  from it's delegated symbol.

2. we also change the oprand of branch instruction from symbol name to qualify symbol name.
for example change
bl .foo
extern .foo
to
bl .foo[PR]
extern .foo[PR]

3. and if there is reference indirect call a function bar.
we also add
  extern .bar[PR]

Reviewers:  Jason liu, Xiangling Liao

Differential Revision: https://reviews.llvm.org/D84765
The file was modifiedllvm/test/CodeGen/PowerPC/aix-xcoff-visibility.ll
The file was modifiedllvm/lib/CodeGen/AsmPrinter/AsmPrinter.cpp
The file was modifiedllvm/test/CodeGen/PowerPC/aix-extern-weak.ll
The file was modifiedllvm/test/CodeGen/PowerPC/aix-xcoff-symbol-rename.ll
The file was modifiedllvm/include/llvm/MC/MCContext.h
The file was modifiedllvm/lib/Target/PowerPC/PPCISelLowering.cpp
The file was modifiedllvm/test/CodeGen/PowerPC/aix64-cc-byval.ll
The file was modifiedllvm/include/llvm/MC/MCSectionXCOFF.h
The file was modifiedllvm/lib/CodeGen/TargetLoweringObjectFileImpl.cpp
The file was modifiedllvm/test/CodeGen/PowerPC/test_call_aix.ll
The file was modifiedllvm/include/llvm/MC/MCSymbolXCOFF.h
The file was modifiedllvm/lib/MC/MCSymbolXCOFF.cpp
The file was modifiedllvm/lib/MC/MCObjectFileInfo.cpp
The file was modifiedllvm/test/CodeGen/PowerPC/aix-xcoff-toc.ll
The file was modifiedllvm/test/CodeGen/PowerPC/aix-external-sym-sdnode-lowering.ll
The file was modifiedllvm/test/CodeGen/PowerPC/aix-sret-param.ll
The file was modifiedllvm/test/CodeGen/PowerPC/aix-cc-byval-mem.ll
The file was modifiedllvm/lib/Target/PowerPC/PPCAsmPrinter.cpp
The file was modifiedllvm/test/CodeGen/PowerPC/aix-cc-abi.ll
The file was modifiedllvm/test/CodeGen/PowerPC/aix-cc-byval.ll
The file was modifiedllvm/test/CodeGen/PowerPC/aix-reference-func-addr-const.ll
The file was modifiedllvm/lib/MC/MCContext.cpp
Commit 1a69f02397ae31a484f640aa2d22dbede9f9e3f1 by tlively
[WebAssembly][NFC] Replace WASM with standard Wasm

The officially specified abbreviation for WebAssembly is Wasm and the
spec explicitly calls out WASM as being an incorrect spelling. This
patch fixes a few comments and error messages to use the
spec-compliant abbreviation.

Differential Revision: https://reviews.llvm.org/D85764
The file was modifiedllvm/lib/Target/WebAssembly/AsmParser/WebAssemblyAsmParser.cpp
The file was modifiedllvm/lib/Target/WebAssembly/WebAssemblyISelLowering.cpp
Commit b9c1bd39d017ab3a480a973234c276ff14b36e58 by aeubanks
[gn build] Fix CodeGenTests

Missing dependency required by https://reviews.llvm.org/D67687.

Fixes
lld-link: error: undefined symbol: public: __cdecl llvm::PipelineTuningOptions::PipelineTuningOptions(void)
>>> referenced by C:\src\llvm-project\llvm\unittests\CodeGen\PassManagerTest.cpp:215
>>>               obj/llvm/unittests/CodeGen/CodeGenTests.PassManagerTest.obj:(private: virtual void __cdecl `anonymous namespace'::PassManagerTest_Basic_Test::TestBody(void))

Differential Revision: https://reviews.llvm.org/D85682
The file was modifiedllvm/utils/gn/secondary/llvm/unittests/CodeGen/BUILD.gn
Commit 9b211a5076310f14081dd257b0cbac6857534bc3 by Louis Dionne
[libunwind] Fix incorrect check for out-of-boundedness

If the personalityIndex (which is 0-based) is equal to the length of
the personality array, we should error out.

rdar://18013273
The file was modifiedlibunwind/src/UnwindCursor.hpp
Commit e74e8b2b6957b4613b0fef2a964d40313074f435 by lebedev.ri
[NFC][InstCombine] Add tests for PHI merging/aggregate reconstruction (PR47060)

We should be able to see that the new aggregate we have produced
is identical to the source aggregate from which we've extracted
the elements that we used to form a new aggregate.

This happens (a lot) in clang C++ exception code on unwind branch.
The file was addedllvm/test/Transforms/InstCombine/phi-aware-aggregate-reconstruction.ll
The file was addedllvm/test/Transforms/InstCombine/aggregate-reconstruction.ll
Commit 254e0abf5be2e98cb7f1fa52617b71f4b94b11a4 by Jonas Devlieghere
[lldb] Fix the last remaining tests not inheriting TCC permissions

After this patch all test should have the inferior inheriting the TCC
permissions from its parent.
The file was modifiedlldb/test/API/commands/expression/weak_symbols/TestWeakSymbols.py
The file was modifiedlldb/test/API/functionalities/target-new-solib-notifications/TestModuleLoadedNotifys.py
The file was modifiedlldb/test/API/types/AbstractBase.py
The file was modifiedlldb/test/API/lang/objc/exceptions/TestObjCExceptions.py
Commit 06d567059e5ad76c0eb1ee85295f76c89ff453a1 by nikita.ppv
[InstSimplify] Respect CanUseUndef in more places

Similar to what we do in IIQ, add an isUndefValue() helper that
checks for undef values while respective CanUseUndef. This makes
it much easier to search for places that don't respect the flag
yet.
The file was modifiedllvm/include/llvm/Analysis/InstructionSimplify.h
The file was modifiedllvm/lib/Analysis/InstructionSimplify.cpp