[all-commits] [llvm/llvm-project] 2f49cd: [AMDGPU] Do not release VGPRs at -O0

5chmidti via All-commits all-commits at lists.llvm.org
Mon Aug 14 01:37:03 PDT 2023


  Branch: refs/heads/release/17.x
  Home:   https://github.com/llvm/llvm-project
  Commit: 2f49cd1a773da7d2227fa7db952f50137a812300
      https://github.com/llvm/llvm-project/commit/2f49cd1a773da7d2227fa7db952f50137a812300
  Author: Jay Foad <jay.foad at amd.com>
  Date:   2023-08-14 (Mon, 14 Aug 2023)

  Changed paths:
    M llvm/lib/Target/AMDGPU/SIInsertWaitcnts.cpp
    A llvm/test/CodeGen/AMDGPU/release-vgprs.mir

  Log Message:
  -----------
  [AMDGPU] Do not release VGPRs at -O0

This was an oversight when the GFX11 early release VGPRs optimization
was reimplemented in D153279.

Sending the DEALLOC_VGPRS message is a performance optimization so there
is no need to do it at -O0. In addition it makes some kinds of post
mortem debugging hard or impossible, since VGPR values are no longer
available to inspect at the s_endpgm instruction.

Differential Revision: https://reviews.llvm.org/D157599

(cherry picked from commit 3091bdb86d55e404866823b64d21fd87c247d893)


  Commit: f3e3c5a922c72c2b483f2ec6dc1dfdb938382203
      https://github.com/llvm/llvm-project/commit/f3e3c5a922c72c2b483f2ec6dc1dfdb938382203
  Author: Tamir Duberstein <tamird at gmail.com>
  Date:   2023-08-14 (Mon, 14 Aug 2023)

  Changed paths:
    M llvm/lib/Target/BPF/BTFDebug.cpp
    A llvm/test/CodeGen/BPF/BTF/incomplete-debuginfo.ll

  Log Message:
  -----------
  [BPF] Don't crash on missing line info

When compiling Rust code we may end up with calls to functions provided
by other code units. Presently this code crashes on a null pointer
dereference - this patch avoids that crash and adds a test.

Reviewed By: ast

Differential Revision: https://reviews.llvm.org/D156446

(cherry picked from commit 055893beacb34441467eb997a270a620c57c138f)


  Commit: 5a600592de19655c4ab41ddff738f7082cce17a2
      https://github.com/llvm/llvm-project/commit/5a600592de19655c4ab41ddff738f7082cce17a2
  Author: Julian Schmidt <44101708+5chmidti at users.noreply.github.com>
  Date:   2023-08-14 (Mon, 14 Aug 2023)

  Changed paths:
    M clang-tools-extra/clang-tidy/tool/run-clang-tidy.py

  Log Message:
  -----------
  [clang-tidy] fix None tmpdir when exporting fixes in run-clang-tidy

Differential https://reviews.llvm.org/D145477 removed the check for `(yaml and args.export_fixes)` in line 303 to skip looking for the `clang-apply-replacements` binary. However, the `tmpdir` variable was set in this true branch when exporting fixes and therefore is `None` when invoking run-clang-tidy with `run-clang-tidy -p . -export-fixes fixes.yaml`.

Reviewed By: PiotrZSL

Differential Revision: https://reviews.llvm.org/D157773

(cherry picked from commit c3da99275a520b73235d975017502876e07e3e8e)


Compare: https://github.com/llvm/llvm-project/compare/8f4dd44097c9...5a600592de19


More information about the All-commits mailing list