[all-commits] [llvm/llvm-project] 098767: Revert changes to llvm-string stdin test and XFAIL...

Maksim Levental via All-commits all-commits at lists.llvm.org
Fri Sep 26 19:18:39 PDT 2025 

  Branch: refs/heads/users/makslevental/checked-gettors
  Home:   https://github.com/llvm/llvm-project
  Commit: 098767cafa18b09a6efcf017dc7092ccc14b288b
      https://github.com/llvm/llvm-project/commit/098767cafa18b09a6efcf017dc7092ccc14b288b
  Author: Mark Danial <mark.danial at ibm.com>
  Date:   2025-09-26 (Fri, 26 Sep 2025)

  Changed paths:
    M llvm/test/tools/llvm-strings/stdin.test

  Log Message:
  -----------
  Revert changes to llvm-string stdin test and XFAIL on AIX (#160821)

Xfail testcase on AIX until echo -n bug is fixed, revert changes made to
testcase


  Commit: 1aefabe5545bbf76a5218b998254033e71ed24f7
      https://github.com/llvm/llvm-project/commit/1aefabe5545bbf76a5218b998254033e71ed24f7
  Author: Jan Svoboda <jan_svoboda at apple.com>
  Date:   2025-09-26 (Fri, 26 Sep 2025)

  Changed paths:
    M clang/lib/Lex/HeaderSearch.cpp

  Log Message:
  -----------
  [clang] Use the VFS to check the system framework marker (#160946)

This PR uses the VFS/`FileManager` to check the system framework marker
instead of going straight to the real file system. This matches the
behavior of other input files of the compiler.


  Commit: 34ebdaf73287e4f0fd9e897d01060071dbc42e99
      https://github.com/llvm/llvm-project/commit/34ebdaf73287e4f0fd9e897d01060071dbc42e99
  Author: Jan Svoboda <jan_svoboda at apple.com>
  Date:   2025-09-26 (Fri, 26 Sep 2025)

  Changed paths:
    M clang/lib/StaticAnalyzer/Frontend/ModelInjector.cpp

  Log Message:
  -----------
  [clang][analyzer] Use the VFS to check model files (#160950)

This PR uses the VFS to check `.model` files in the Clang static
analyzer to match the compiler's behavior for other input files.


  Commit: 7b3d28573c1c3ae74fc829d659a6a99c02d961f9
      https://github.com/llvm/llvm-project/commit/7b3d28573c1c3ae74fc829d659a6a99c02d961f9
  Author: Jan Svoboda <jan_svoboda at apple.com>
  Date:   2025-09-26 (Fri, 26 Sep 2025)

  Changed paths:
    M clang/lib/CodeGen/CodeGenModule.cpp

  Log Message:
  -----------
  [clang] Use the VFS to get the unique file ID (#160936)

This PR uses the VFS to get the unique file ID when printing
externalized decls in CUDA instead of going straight to the real file
system. This matches the behavior of other input files of the compiler.


  Commit: 220ad03d5b28dbd83dbb90896a7eb55b74004818
      https://github.com/llvm/llvm-project/commit/220ad03d5b28dbd83dbb90896a7eb55b74004818
  Author: Jan Svoboda <jan_svoboda at apple.com>
  Date:   2025-09-26 (Fri, 26 Sep 2025)

  Changed paths:
    M clang/lib/CodeGen/CGOpenMPRuntime.cpp

  Log Message:
  -----------
  [clang] Use the VFS to get the OpenMP entry info (#160935)

This PR uses the VFS to get the OpenMP entry info instead of going
straight to the real file system. This matches the behavior of other
input files of the compiler.


  Commit: f9065fce8fd2bb664ed8dcb7582432b83c2114c4
      https://github.com/llvm/llvm-project/commit/f9065fce8fd2bb664ed8dcb7582432b83c2114c4
  Author: Paul Kirth <paulkirth at google.com>
  Date:   2025-09-26 (Fri, 26 Sep 2025)

  Changed paths:
    M llvm/lib/Support/Mustache.cpp

  Log Message:
  -----------
  [llvm][mustache] Avoid excessive hash lookups in EscapeStringStream (#160166)

The naive char-by-char lookup performed OK, but we can skip ahead to the
next match, avoiding all the extra hash lookups in the key map. Likely
there is a faster method than this, but its already a 42% win in the
BM_Mustache_StringRendering/Escaped benchmark, and an order of magnitude
improvement for BM_Mustache_LargeOutputString.

| Benchmark | Before (ns) | After (ns) | Speedup |
| :--- | ---: | ---: | ---: |
| `StringRendering/Escaped` | 29,440,922 | 16,583,603 | ~44% |
| `LargeOutputString` | 15,139,251 | 929,891 | ~94% |
| `HugeArrayIteration` | 102,148,245 | 95,943,960 | ~6% |
| `PartialsRendering` | 308,330,014 | 303,556,563 | ~1.6% |

Unreported benchmarks, like those for parsing, had no significant
change.


  Commit: 37e7ad184d002db15f72771938755580433cf96d
      https://github.com/llvm/llvm-project/commit/37e7ad184d002db15f72771938755580433cf96d
  Author: Aiden Grossman <aidengrossman at google.com>
  Date:   2025-09-26 (Fri, 26 Sep 2025)

  Changed paths:
    M libcxx/utils/libcxx/test/format.py

  Log Message:
  -----------
  [libcxx] Add local %T substitution

This patch adds a %T substitution directly into the libc++ test format.
This ensures that the libc++ test format will continue to work when we
remove support for %T in llvm lit.

Reviewers: #reviewers-libcxx, ldionne, philnik777

Pull Request: https://github.com/llvm/llvm-project/pull/160026


  Commit: f8d547fa8a038d0adf5c07708d11bbf1d8db3100
      https://github.com/llvm/llvm-project/commit/f8d547fa8a038d0adf5c07708d11bbf1d8db3100
  Author: Aiden Grossman <aidengrossman at google.com>
  Date:   2025-09-26 (Fri, 26 Sep 2025)

  Changed paths:
    M llvm/utils/lit/tests/Inputs/shtest-shell/valid-shell.txt

  Log Message:
  -----------
  [lit] Remove %T from tests

This patch removes %T from tests in preparation for removing the
substitution completely. Splitting this off from the actual removal to
make it easier to revert the removal of %T in case I missed something.

Reviewers: RKSimon, jh7370, cmtice, petrhosek, ilovepi, pogo59

Reviewed By: cmtice, RKSimon, jh7370

Pull Request: https://github.com/llvm/llvm-project/pull/160027


  Commit: 597f93d36b035faeb63f4ba0d61a8b8e25eddaab
      https://github.com/llvm/llvm-project/commit/597f93d36b035faeb63f4ba0d61a8b8e25eddaab
  Author: Matt Arsenault <Matthew.Arsenault at amd.com>
  Date:   2025-09-27 (Sat, 27 Sep 2025)

  Changed paths:
    M llvm/lib/Target/AMDGPU/SIFoldOperands.cpp
    M llvm/test/CodeGen/AMDGPU/fold-imm-copy-agpr.mir
    M llvm/test/CodeGen/AMDGPU/fold-imm-copy.mir
    A llvm/test/CodeGen/AMDGPU/fold-operands-frame-index-agpr.mir

  Log Message:
  -----------
  AMDGPU: Check if immediate is legal for av_mov_b32_imm_pseudo (#160819)

This is primarily to avoid folding a frame index materialized
into an SGPR into the pseudo; this would end up looking like:
  %sreg = s_mov_b32 %stack.0
  %av_32 = av_mov_b32_imm_pseudo %sreg

Which is not useful.

Match the check used for the b64 case. This is limited to the
pseudo to avoid regression due to gfx908's special case - it
is expecting to pass here with v_accvgpr_write_b32 for illegal
cases, and stay in the intermediate state with an sgpr input.

This avoids regressions in a future patch.


  Commit: e925e528d9750a424f1a3c8519aae9e0288d13a4
      https://github.com/llvm/llvm-project/commit/e925e528d9750a424f1a3c8519aae9e0288d13a4
  Author: Sam Clegg <sbc at chromium.org>
  Date:   2025-09-26 (Fri, 26 Sep 2025)

  Changed paths:
    M lld/test/wasm/mutable-global-exports.s
    M lld/wasm/Writer.cpp

  Log Message:
  -----------
  [lld][WebAssembly] Fix check for implicitly exported mutable globals (#160966)

This check is designed to avoid exporting mutable globals in the case
when mutable globals are not available. However, it was being applied in
all cases even when mutable globals was enabled.

This error is particularly bad since mutable-globals have been enabled
in default CPU for a while now, meaning that this condition should not
be firing in the wild very often.


  Commit: 727ce02653c29467b861a6ede0c925f3fdb8a6cb
      https://github.com/llvm/llvm-project/commit/727ce02653c29467b861a6ede0c925f3fdb8a6cb
  Author: Ellis Hoag <ellis.sparky.hoag at gmail.com>
  Date:   2025-09-26 (Fri, 26 Sep 2025)

  Changed paths:
    M lld/MachO/SyntheticSections.cpp
    A lld/test/MachO/order-file-cstring.s
    R lld/test/MachO/ordre-file-cstring.s

  Log Message:
  -----------
  [lld][macho] Rename test to fix spelling (#160938)

The test name had a typo from
https://github.com/llvm/llvm-project/pull/140307. Fix it.

I realized cstring ordering is not supported when string deduplication
is turned off. We could easily call `buildCStringPriorities()` in
`CStringSection::finalizeContents()`, but I worry it might harm build
performance since it creates multiple vectors and searches though maps.
If users are not deduplicating strings, they probably won't care to
order them, but it would be good to support this.


  Commit: 2be906b25581eebbd6607a5d99943251617622cb
      https://github.com/llvm/llvm-project/commit/2be906b25581eebbd6607a5d99943251617622cb
  Author: ShihPo Hung <shihpo.hung at sifive.com>
  Date:   2025-09-26 (Fri, 26 Sep 2025)

  Changed paths:
    M llvm/include/llvm/Analysis/TargetTransformInfo.h
    M llvm/include/llvm/Analysis/TargetTransformInfoImpl.h
    M llvm/include/llvm/CodeGen/BasicTTIImpl.h
    M llvm/lib/Analysis/TargetTransformInfo.cpp
    M llvm/lib/Target/RISCV/RISCVISelLowering.cpp
    M llvm/lib/Target/RISCV/RISCVISelLowering.h
    M llvm/lib/Target/RISCV/RISCVTargetTransformInfo.cpp
    M llvm/lib/Target/RISCV/RISCVTargetTransformInfo.h
    M llvm/test/Analysis/CostModel/RISCV/vp-intrinsics.ll

  Log Message:
  -----------
  Revert "[TTI][RISCV] Add cost modelling for intrinsic vp.load.ff (#160470)"

This reverts commit aa08b1a9963f33ded658d3ee655429e1121b5212.


  Commit: be6c5d0663790817291cab90248ada0117720d6a
      https://github.com/llvm/llvm-project/commit/be6c5d0663790817291cab90248ada0117720d6a
  Author: co63oc <co63 at 163.com>
  Date:   2025-09-26 (Fri, 26 Sep 2025)

  Changed paths:
    M compiler-rt/lib/fuzzer/FuzzerCorpus.h
    M compiler-rt/lib/fuzzer/FuzzerDriver.cpp
    M compiler-rt/lib/fuzzer/FuzzerExtFunctionsWindows.cpp
    M compiler-rt/lib/fuzzer/FuzzerLoop.cpp
    M compiler-rt/lib/fuzzer/FuzzerMutate.cpp

  Log Message:
  -----------
  [NFC] [compiler-rt] fix typos (#160803)

fix typos


  Commit: 7ff6973f1ae91985bb6b5faa8658e81dbaadbd25
      https://github.com/llvm/llvm-project/commit/7ff6973f1ae91985bb6b5faa8658e81dbaadbd25
  Author: Aiden Grossman <aidengrossman at google.com>
  Date:   2025-09-26 (Fri, 26 Sep 2025)

  Changed paths:
    M llvm/docs/CommandGuide/lit.rst
    M llvm/docs/ReleaseNotes.md
    M llvm/utils/lit/lit/TestRunner.py
    A llvm/utils/lit/tests/Inputs/shtest-shell/capital-t-error-message.txt
    M llvm/utils/lit/tests/shtest-shell.py

  Log Message:
  -----------
  [lit] Remove support for %T

This patch removes support for %T from llvm-lit. For now we mark the
test unresolved and add an error message noting the substitution is
deprecated. This is exactly the same as the error handling for other
substitution failures. We intend to remove support for the nice error
message once 22 branches as users should have moved over by the they are
upgrading to v23.

Reviewers: petrhosek, jh7370, ilovepi, pogo59, cmtice

Reviewed By: cmtice, jh7370, ilovepi

Pull Request: https://github.com/llvm/llvm-project/pull/160028


  Commit: 76b167e53db8981a5745e9280a858937fa7fcdc5
      https://github.com/llvm/llvm-project/commit/76b167e53db8981a5745e9280a858937fa7fcdc5
  Author: Maksim Levental <maksim.levental at gmail.com>
  Date:   2025-09-26 (Fri, 26 Sep 2025)

  Changed paths:
    M clang/lib/CodeGen/CGOpenMPRuntime.cpp
    M clang/lib/CodeGen/CodeGenModule.cpp
    M clang/lib/Lex/HeaderSearch.cpp
    M clang/lib/StaticAnalyzer/Frontend/ModelInjector.cpp
    M compiler-rt/lib/fuzzer/FuzzerCorpus.h
    M compiler-rt/lib/fuzzer/FuzzerDriver.cpp
    M compiler-rt/lib/fuzzer/FuzzerExtFunctionsWindows.cpp
    M compiler-rt/lib/fuzzer/FuzzerLoop.cpp
    M compiler-rt/lib/fuzzer/FuzzerMutate.cpp
    M libcxx/utils/libcxx/test/format.py
    M lld/MachO/SyntheticSections.cpp
    A lld/test/MachO/order-file-cstring.s
    R lld/test/MachO/ordre-file-cstring.s
    M lld/test/wasm/mutable-global-exports.s
    M lld/wasm/Writer.cpp
    M llvm/docs/CommandGuide/lit.rst
    M llvm/docs/ReleaseNotes.md
    M llvm/include/llvm/Analysis/TargetTransformInfo.h
    M llvm/include/llvm/Analysis/TargetTransformInfoImpl.h
    M llvm/include/llvm/CodeGen/BasicTTIImpl.h
    M llvm/lib/Analysis/TargetTransformInfo.cpp
    M llvm/lib/Support/Mustache.cpp
    M llvm/lib/Target/AMDGPU/SIFoldOperands.cpp
    M llvm/lib/Target/RISCV/RISCVISelLowering.cpp
    M llvm/lib/Target/RISCV/RISCVISelLowering.h
    M llvm/lib/Target/RISCV/RISCVTargetTransformInfo.cpp
    M llvm/lib/Target/RISCV/RISCVTargetTransformInfo.h
    M llvm/test/Analysis/CostModel/RISCV/vp-intrinsics.ll
    M llvm/test/CodeGen/AMDGPU/fold-imm-copy-agpr.mir
    M llvm/test/CodeGen/AMDGPU/fold-imm-copy.mir
    A llvm/test/CodeGen/AMDGPU/fold-operands-frame-index-agpr.mir
    M llvm/test/tools/llvm-strings/stdin.test
    M llvm/utils/lit/lit/TestRunner.py
    A llvm/utils/lit/tests/Inputs/shtest-shell/capital-t-error-message.txt
    M llvm/utils/lit/tests/Inputs/shtest-shell/valid-shell.txt
    M llvm/utils/lit/tests/shtest-shell.py

  Log Message:
  -----------
  Merge branch 'main' into users/makslevental/checked-gettors


Compare: https://github.com/llvm/llvm-project/compare/bcc8bcad3b7a...76b167e53db8

To unsubscribe from these emails, change your notification settings at https://github.com/llvm/llvm-project/settings/notifications

More information about the All-commits mailing list