SuccessChanges

Summary

  1. [buildbot] Use paths instead of env vars for ml bots (details)
Commit d99cf8c14b7b1552c869a019e8533e1e3363aa4f by mtrofin
[buildbot] Use paths instead of env vars for ml bots

Summary: Having trouble with env vars, so using hard-coded paths for now

Reviewers: davidxl, gkistanova

Differential Revision: https://reviews.llvm.org/D82482
The file was modifiedbuildbot/osuosl/master/config/builders.py (diff)

Summary

  1. Disable a flaky lldb-vscode test on aarch64 (details)
  2. [X86] Rename O3-pipeline.ll to opt-pipeline.ll and add O1/O2 command lines (details)
  3. [NVPTX] Fix for NVPTX module asm regression (details)
  4. [llvm-objcopy] Update help messages (details)
  5. [OpenMP][Docs] Mark TR8 `present` as claimed in docs (details)
  6. Local.h - reduce includes to forward declarations. NFC. (details)
  7. InstCombineInternal.h - reduce AliasAnalysis.h include to forward declaration. NFC. (details)
  8. [OpenMP][NFC] Remove hard-coded line numbers from test (details)
  9. [ObjC] Copy a block to the heap if it is passed as a variadic argument (details)
  10. [FileCheck][NFC] Fix typo in test comment (details)
  11. [docs] Fix typo (details)
  12. [SLP] Limit GEP lists based on width of index computation. (details)
  13. [Apple Silicon] Initial support for Rosetta (details)
Commit d4ef569577625d50dfe01ab54c7b9995db3e013f by waltermelon
Disable a flaky lldb-vscode test on aarch64

Summary:
These tests isflaky only on this arch for some reason. It's testing important features and is not flaky on x86_64, so I'll investigate this arm issue separatedly.

A flaky run:
http://lab.llvm.org:8011/builders/lldb-aarch64-ubuntu/builds/5540/steps/test/logs/stdio

Diff that created those tests:
https://reviews.llvm.org/D81978
The file was modifiedlldb/test/API/tools/lldb-vscode/launch/TestVSCode_launch.py
Commit 1a4f8889800a8c80819602725c5ce509ded2d451 by craig.topper
[X86] Rename O3-pipeline.ll to opt-pipeline.ll and add O1/O2 command lines

Eric Cristopher asked me about possibly disabling some passes at
-O1/Og. Figured a good first step was to test all the pipelines.
They all appear to be the same for now. Hoping we can use FileCheck
prefixes for differences to avoid repeating the contents 3 times.
The file was addedllvm/test/CodeGen/X86/opt-pipeline.ll
The file was removedllvm/test/CodeGen/X86/O3-pipeline.ll
Commit af5e61bf4fd1bf865f90beab0227ff4901246eeb by tra
[NVPTX] Fix for NVPTX module asm regression

Currently module asm ends up emitted twice and at the wrong place in the PTX.
This patch moves module asm generation into emitStartOfAsmFile() which puts at
the correct location in the generated PTX.

Differential Revision: https://reviews.llvm.org/D82280
The file was modifiedllvm/test/CodeGen/NVPTX/module-inline-asm.ll
The file was modifiedllvm/lib/Target/NVPTX/NVPTXAsmPrinter.cpp
The file was modifiedllvm/lib/Target/NVPTX/NVPTXAsmPrinter.h
Commit ca133cdecfeacf9a36f7f4b547239ca8b799641c by alexshap
[llvm-objcopy] Update help messages

This diff updates the help messages for llvm-objcopy, llvm-strip and
llvm-install-name-tool.

Patch by Sameer Arora!

Test plan: make check-all

Differential revision: https://reviews.llvm.org/D81907
The file was modifiedllvm/tools/llvm-objcopy/CopyConfig.cpp
The file was modifiedllvm/test/tools/llvm-objcopy/MachO/install-name-tool-help-message.test
The file was modifiedllvm/test/tools/llvm-objcopy/ELF/help-message.test
Commit 3fa666b883625a678cfcfd9ad96b2daabcef09e8 by jdenny.ornl
[OpenMP][Docs] Mark TR8 `present` as claimed in docs
The file was modifiedclang/docs/OpenMPSupport.rst
Commit a53dddb3e95de0fb80d280bd625a1ebc04315d51 by llvm-dev
Local.h - reduce includes to forward declarations. NFC.

Fix implicit include dependencies in source files and replace legacy AliasAnalysis typedef with AAResults where necessary.
The file was modifiedllvm/lib/Transforms/Scalar/SCCP.cpp
The file was modifiedllvm/include/llvm/Transforms/Utils/Local.h
The file was modifiedllvm/lib/CodeGen/HardwareLoops.cpp
The file was modifiedllvm/lib/Target/AArch64/AArch64StackTagging.cpp
The file was modifiedllvm/lib/Transforms/Utils/FlattenCFG.cpp
Commit 6c6adde84f31fb83ba35dc08a8420afb7ce7e093 by llvm-dev
InstCombineInternal.h - reduce AliasAnalysis.h include to forward declaration. NFC.

Fix implicit include dependencies in source files and replace legacy AliasAnalysis typedef with AAResults where necessary.
The file was modifiedllvm/lib/Transforms/InstCombine/InstCombineLoadStoreAlloca.cpp
The file was modifiedllvm/lib/Transforms/InstCombine/InstCombineInternal.h
Commit 01ddb2a7b044f697a15043e47acdb93e2825809a by jdenny.ornl
[OpenMP][NFC] Remove hard-coded line numbers from test

Otherwise, it's painful to insert new code.  There are many existing
examples in the same test file where the line numbers are not
hard-coded.

I intend to do the same for several other OpenMP tests, but I want to
be sure there are no objections before I spend time on it.

Reviewed By: jdoerfert

Differential Revision: https://reviews.llvm.org/D82224
The file was modifiedclang/test/OpenMP/target_map_codegen.cpp
Commit cdd6a2788caced6b377af818154138d9983dba5f by Akira
[ObjC] Copy a block to the heap if it is passed as a variadic argument

Call maybeExtendBlockObject in DefaultVariadicArgumentPromotion so that
the block is copied to the heap when it is passed as a variadic argument
to any calls, not only to C function calls.

rdar://problem/64201532
The file was modifiedclang/test/CodeGenObjC/arc-blocks.m
The file was modifiedclang/lib/Sema/SemaExpr.cpp
Commit ecb098c6debd13a8fbe5c730a5c717b627ee27cd by jdenny.ornl
[FileCheck][NFC] Fix typo in test comment
The file was modifiedllvm/test/FileCheck/dump-input-enable.txt
Commit d65cdb498f901d7ce03dbaa1831c629400ebb422 by Vedant Kumar
[docs] Fix typo
The file was modifiedllvm/docs/HowToUpdateDebugInfo.rst
Commit 35bb9bfbb099eb78dbed8f96d7cd7f5506d6584f by flo
[SLP] Limit GEP lists based on width of index computation.

D68667 introduced a tighter limit to the number of GEPs to simplify
together. The limit was based on the vector element size of the pointer,
but the pointers themselves are not actually put in vectors.

IIUC we try to vectorize the index computations here, so we should base
the limit on the vector element size of the computation of the index.

This restores the test regression on AArch64 and also restores the
vectorization for a important pattern in SPEC2006/464.h264ref on
AArch64 (@test_i16_extend). We get a large benefit from doing a single
load up front and then processing the index computations in vectors.

Note that we could probably even further improve the AArch64 codegen, if
we would do zexts to i32 instead of i64 for the sub operands and then do
a single vector sext on the result of the subtractions. AArch64 provides
dedicated vector instructions to do so. Sketch of proof in Alive:
https://alive2.llvm.org/ce/z/A4xYAB

Reviewers: craig.topper, RKSimon, xbolva00, ABataev, spatel

Reviewed By: ABataev, spatel

Differential Revision: https://reviews.llvm.org/D82418
The file was modifiedllvm/test/Transforms/SLPVectorizer/AArch64/getelementptr.ll
The file was modifiedllvm/lib/Transforms/Vectorize/SLPVectorizer.cpp
Commit fd19ddb8f2a2b082f492fc59f7f360adf3495701 by ditaliano
[Apple Silicon] Initial support for Rosetta

Translated processes talk with a different debugserver, shipped with
macOS 11. This patch detects whether a process is translated and
attaches to the correct debugserver implementation.
It's the first patch of a series. Tested on the lldb test suite.

Differential Revision:  https://reviews.llvm.org/D82491
The file was modifiedlldb/source/Plugins/Process/gdb-remote/ProcessGDBRemote.cpp