[all-commits] [llvm/llvm-project] aa7e4b: [BOLT] Fix an unused variable warning

Vitaly Buka via All-commits all-commits at lists.llvm.org
Wed Mar 20 19:59:05 PDT 2024


  Branch: refs/heads/users/vitalybuka/spr/main.codegen-add-default-lowering-for-llvmallowruntimeubsancheck
  Home:   https://github.com/llvm/llvm-project
  Commit: aa7e4ba3cad0e00dd37d4baca680ed1633bbdb70
      https://github.com/llvm/llvm-project/commit/aa7e4ba3cad0e00dd37d4baca680ed1633bbdb70
  Author: Kazu Hirata <kazu at google.com>
  Date:   2024-03-20 (Wed, 20 Mar 2024)

  Changed paths:
    M bolt/lib/Profile/BoltAddressTranslation.cpp

  Log Message:
  -----------
  [BOLT] Fix an unused variable warning

This patch fixes:

  bolt/lib/Profile/BoltAddressTranslation.cpp:26:12: error: unused
  variable 'HotFuncAddress' [-Werror,-Wunused-variable]


  Commit: 893717446bbc8f31427b581af5fbaf4154b63402
      https://github.com/llvm/llvm-project/commit/893717446bbc8f31427b581af5fbaf4154b63402
  Author: Joseph Huber <huberjn at outlook.com>
  Date:   2024-03-20 (Wed, 20 Mar 2024)

  Changed paths:
    M libc/CMakeLists.txt

  Log Message:
  -----------
  [libc] Add an option to always build GPU loader utilities (#86040)

Summary:
Right now it's difficult to build these utilities standalone becayse
they're keyed off of the other GPU handling. if someone wants to *just*
build these utilities it's not possible without setting up the runtimes
build. Since we can't just build these by default add an option to
enable it.

We can't just use the handling like LIBC_HDRGEN does because this is
only for the GPU build, which isn't fully set up until way later. So
this is probably the easiest way to just allow people to build these
tools even without a GPU build setup.


  Commit: 3cd988914e53c4c94e48edd4b6bc7b97b2dd4b49
      https://github.com/llvm/llvm-project/commit/3cd988914e53c4c94e48edd4b6bc7b97b2dd4b49
  Author: Alexander Yermolovich <43973793+ayermolo at users.noreply.github.com>
  Date:   2024-03-20 (Wed, 20 Mar 2024)

  Changed paths:
    M bolt/test/X86/dwarf5-label-low-pc.s

  Log Message:
  -----------
  [BOLT][DWARF] Fix Test (#86042)

Test was not actually checking bolt binary, and had extra POSTCHECK-NEXT
lines.


  Commit: 71defe40b7df18508d63fb1b1233324e8a28688f
      https://github.com/llvm/llvm-project/commit/71defe40b7df18508d63fb1b1233324e8a28688f
  Author: Thurston Dang <thurston at google.com>
  Date:   2024-03-20 (Wed, 20 Mar 2024)

  Changed paths:
    M compiler-rt/lib/sanitizer_common/sanitizer_posix_libcdep.cpp

  Log Message:
  -----------
  [sanitizer_common] Suppress warning of cast from SignalHandlerType to sa_sigaction_t (#86046)

Some buildbots (e.g.,
https://lab.llvm.org/buildbot/#/builders/18/builds/16061/steps/10/logs/stdio)
have recently started complaining about
```
cast from 'SignalHandlerType' (aka 'void (*)(int, void *, void *)') to 'sa_sigaction_t' (aka 'void (*)(int, siginfo_t *, void *)') converts to incompatible function type [-Werror,-Wcast-function-type-strict]
  219 |   sigact.sa_sigaction = (sa_sigaction_t)handler;
  ```
This patch does an intermediate cast to `'(void (*) (void))'` to suppress the warning.

N.B. SignalHandlerType has `'void*'` instead of `'siginfo_t*'` because it is typedef'ed in sanitizer_common/sanitizer_common.h, which does not have access to the header (signal.h) that defines siginfo_t; we therefore cannot fix SignalHandlerType.


  Commit: 631248dcd26fdec772cedb569be94ff8f12d0901
      https://github.com/llvm/llvm-project/commit/631248dcd26fdec772cedb569be94ff8f12d0901
  Author: hstk30-hw <hanwei62 at huawei.com>
  Date:   2024-03-21 (Thu, 21 Mar 2024)

  Changed paths:
    M clang/lib/CodeGen/Targets/X86.cpp
    A clang/test/CodeGenCXX/x86_64-vaarg.cpp

  Log Message:
  -----------
  [X86_64] fix empty structure vaarg in c++ (#77907)

SizeInBytes of empty structure is 0 in C, while 1 in C++. And empty
structure argument of the function is ignored in X86_64 backend.As a
result, the value of variable arguments in C++ is incorrect. fix #77036

Co-authored-by: Longsheng Mou <moulongsheng at huawei.com>


  Commit: 35a66f965c0ea3b806b2b1736bfe4e6eb61d3613
      https://github.com/llvm/llvm-project/commit/35a66f965c0ea3b806b2b1736bfe4e6eb61d3613
  Author: Freddy Ye <freddy.ye at intel.com>
  Date:   2024-03-21 (Thu, 21 Mar 2024)

  Changed paths:
    A llvm/test/CodeGen/X86/domain-reassignment-ndd.mir

  Log Message:
  -----------
  Precommit test for #85737 (#86056)

Copied from llvm/test/CodeGen/X86/domain-reassignment.mir


  Commit: 44a81af510801edce842e9574ec4d52cc7bd0ae9
      https://github.com/llvm/llvm-project/commit/44a81af510801edce842e9574ec4d52cc7bd0ae9
  Author: paperchalice <liujunchang97 at outlook.com>
  Date:   2024-03-21 (Thu, 21 Mar 2024)

  Changed paths:
    M llvm/test/Transforms/LoopIdiom/AArch64/byte-compare-index.ll

  Log Message:
  -----------
  [AArch64] Run LoopSimplifyPass in byte-compare-index.ll (#86053)

Make this test case work on both new and legacy pass manager. See also
#85215


  Commit: 0d08282310e4007dfb748132e5c196765b1ffcd2
      https://github.com/llvm/llvm-project/commit/0d08282310e4007dfb748132e5c196765b1ffcd2
  Author: Michael Liao <michael.hliao at gmail.com>
  Date:   2024-03-20 (Wed, 20 Mar 2024)

  Changed paths:
    M mlir/lib/Dialect/XeGPU/IR/CMakeLists.txt

  Log Message:
  -----------
  [MLIR][XeGPU] Fix shared build. NFC


  Commit: 0e3fbfd1e106dd027aab9ea4a9a6f116d05a0987
      https://github.com/llvm/llvm-project/commit/0e3fbfd1e106dd027aab9ea4a9a6f116d05a0987
  Author: Thurston Dang <thurston at google.com>
  Date:   2024-03-21 (Thu, 21 Mar 2024)

  Changed paths:
    M compiler-rt/lib/sanitizer_common/sanitizer_posix_libcdep.cpp

  Log Message:
  -----------
  Revert "[sanitizer_common] Suppress warning of cast from SignalHandlerType to sa_sigaction_t"

This reverts commit 9d79589e7c8b728a592a4b6b3dee53ac471d7946
because it failed to suppress the warning.


  Commit: 5c95484061a58250de7e5abe150c6ebb25898523
      https://github.com/llvm/llvm-project/commit/5c95484061a58250de7e5abe150c6ebb25898523
  Author: Vitaly Buka <vitalybuka at google.com>
  Date:   2024-03-20 (Wed, 20 Mar 2024)

  Changed paths:
    M llvm/test/Analysis/AliasSet/intrinsics.ll

  Log Message:
  -----------
  [Analysis] Use implicit-check-not in test


  Commit: 3a8dfd48405f4293013f71e54bf1f851d07633d2
      https://github.com/llvm/llvm-project/commit/3a8dfd48405f4293013f71e54bf1f851d07633d2
  Author: Vitaly Buka <vitalybuka at google.com>
  Date:   2024-03-20 (Wed, 20 Mar 2024)

  Changed paths:
    M bolt/lib/Profile/BoltAddressTranslation.cpp
    M bolt/test/X86/dwarf5-label-low-pc.s
    M clang/lib/CodeGen/Targets/X86.cpp
    A clang/test/CodeGenCXX/x86_64-vaarg.cpp
    M libc/CMakeLists.txt
    M llvm/test/Analysis/AliasSet/intrinsics.ll
    A llvm/test/CodeGen/X86/domain-reassignment-ndd.mir
    M llvm/test/Transforms/LoopIdiom/AArch64/byte-compare-index.ll
    M mlir/lib/Dialect/XeGPU/IR/CMakeLists.txt

  Log Message:
  -----------
  [𝘀𝗽𝗿] changes introduced through rebase

Created using spr 1.3.4

[skip ci]


Compare: https://github.com/llvm/llvm-project/compare/4ce5c1a1785c...3a8dfd48405f

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