SuccessChanges

Summary

  1. [test] Factor out creation of copy of SCC Nodes into function (details)
  2. [DSE] Add more tests for read clobber location (NFC) (details)
  3. [gn build] (manually) merge f4c8b8031800 (details)
  4. [mlir][ArmSVE] Add documentation generation (details)
  5. [AttrDocs] document always_inline (details)
  6. [OpenMP][NFC] Provide a new remark and documentation (details)
  7. [gn build] Link with -Wl,--gdb-index when linking with LLD (details)
  8. clang-cl: Remove /Zd flag (details)
Commit 7529fab602c728d12c387e5eb5bbced1ec139dbd by aeubanks
[test] Factor out creation of copy of SCC Nodes into function

Reviewed By: rnk

Differential Revision: https://reviews.llvm.org/D93434
The file was modifiedllvm/unittests/Analysis/CGSCCPassManagerTest.cpp
Commit 1b84934f908d7ab04443f2d442a19d058aadc2ed by nikita.ppv
[DSE] Add more tests for read clobber location (NFC)
The file was addedllvm/test/Transforms/DeadStoreElimination/MSSA/scoped-noalias.ll
The file was modifiedllvm/test/Transforms/DeadStoreElimination/MSSA/overlap.ll
Commit 85ffbe5d6a0d6bf5ad651f415ca01dc3e3c4dc76 by thakis
[gn build] (manually) merge f4c8b8031800
The file was modifiedllvm/utils/gn/secondary/llvm/include/llvm/Frontend/OpenMP/BUILD.gn
Commit 106e66f3f555c8f887e82c5f04c3e77bdaf345e8 by ajcbik
[mlir][ArmSVE] Add documentation generation

Adds missing cmake command to generate documentation for ArmSVE
Dialect.

Reviewed By: aartbik

Differential Revision: https://reviews.llvm.org/D93465
The file was modifiedmlir/include/mlir/Dialect/ArmSVE/CMakeLists.txt
Commit e75fec2b238f0e26cfb7645f2208baebe3440d41 by ndesaulniers
[AttrDocs] document always_inline

GNU documentaion for always_inline:
https://gcc.gnu.org/onlinedocs/gcc/Inline.html

GNU documentation for function attributes:
https://gcc.gnu.org/onlinedocs/gcc/Common-Function-Attributes.html

Microsoft documentation for __force_inline:
https://docs.microsoft.com/en-us/cpp/cpp/inline-functions-cpp

Reviewed By: ojeda

Differential Revision: https://reviews.llvm.org/D68410
The file was modifiedclang/include/clang/Basic/Attr.td
The file was modifiedclang/include/clang/Basic/AttrDocs.td
Commit 994bb6eb7d01db1d9461e54d17a63af2ba1af2c9 by johannes
[OpenMP][NFC] Provide a new remark and documentation

If a GPU function is externally reachable we give up trying to find the
(unique) kernel it is called from. This can hinder optimizations. Emit a
remark and explain mitigation strategies.

Reviewed By: tianshilei1992

Differential Revision: https://reviews.llvm.org/D93439
The file was modifiedopenmp/docs/remarks/OptimizationRemarks.rst
The file was modifiedclang/test/OpenMP/remarks_parallel_in_target_state_machine.c
The file was modifiedllvm/lib/Transforms/IPO/OpenMPOpt.cpp
The file was modifiedclang/test/OpenMP/remarks_parallel_in_multiple_target_state_machines.c
Commit 7e33fd9ce2d8f94bb7195c417426620037804834 by thakis
[gn build] Link with -Wl,--gdb-index when linking with LLD

For full-debug-info (is_debug=true / symbol_level=2 builds), this makes
linking 15% slower, but gdb startup 1500% faster (for lld: link time
3.9s->4.4s, gdb load time >30s->2s).

For link time, I ran

    bench.py -o {noindex,index}.txt \
        sh -c 'rm out/gn/bin/lld && ninja -C out/gn lld'

and then `ministat noindex.txt index.txt`:

```
x noindex.txt
+ index.txt
    N           Min           Max        Median           Avg        Stddev
x   5      3.784461     4.0200169     3.8452811     3.8754988   0.089902595
+   5       4.32496     4.6058481     4.3361208     4.4141198    0.12288267
Difference at 95.0% confidence
0.538621 +/- 0.15702
13.8981% +/- 4.05161%
(Student's t, pooled s = 0.107663)
```

For gdb load time I loaded the crash in PR48392 with

    gdb -ex r --args ../out/gn/bin/ld64.lld.darwinnew @response.txt

and just stopped the time until the crash got displayed with a stopwatch
a few times. So the speedup there is less precise, but it's so
pronounced that that's ok (loads ~instantly with the patch, takes a very
long time without it).

Only doing this for LLD because I haven't tried it with other linkers.

Differential Revision: https://reviews.llvm.org/D92844
The file was modifiedllvm/utils/gn/build/BUILD.gn
Commit 49c248bd62a3cb2f0e7e3991ee0190943bd2bbec by thakis
clang-cl: Remove /Zd flag

cl.exe doesn't understand Zd (in either MSVC 2017 or 2019), so neiter
should we. It used to do the same as `-gline-tables-only` which is
exposed as clang-cl flag as well, so if you want this behavior, use
`gline-tables-only`. That makes it clear that it's a clang-cl-only flag
that won't work with cl.exe.

Motivated by the discussion in D92958.

Differential Revision: https://reviews.llvm.org/D93458
The file was modifiedclang/lib/Driver/ToolChains/MSVC.cpp
The file was modifiedclang/docs/ReleaseNotes.rst
The file was modifiedclang/lib/Driver/ToolChains/Clang.cpp
The file was modifiedclang/include/clang/Driver/Options.td
The file was modifiedclang/test/Driver/cl-options.c

Summary

  1. Attempt to fix Python 3 issues in annotated scripts (details)
  2. Put back missing strip in annotated script (details)
  3. Fix next Python 3 Unicode issue, the gift that will keep giving for the next 20 years (details)
Commit 4cf14ad11aa3a23f67b0f7561edb7d570bf7aacd by rnk
Attempt to fix Python 3 issues in annotated scripts
The file was modifiedzorg/buildbot/builders/annotated/annotated_builder.py
Commit 0ef61a790bbe832d236ebbc40d687d0cc21e8e81 by rnk
Put back missing strip in annotated script
The file was modifiedzorg/buildbot/builders/annotated/annotated_builder.py
Commit b474bb2325bdafa127d6a1504e75b7f7707f879b by rnk
Fix next Python 3 Unicode issue, the gift that will keep giving for the next 20 years
The file was modifiedzorg/buildbot/builders/annotated/annotated_builder.py