[clang] Enclose code in nondebug in #ifndef NDEBUG/#endif (PR #142189)

2025-05-30 Thread LLVM Continuous Integration via cfe-commits
llvm-ci wrote: LLVM Buildbot has detected a new failure on builder `openmp-offload-amdgpu-runtime-2` running on `rocm-worker-hw-02` while building `clang` at step 6 "test-openmp". Full details are available at: https://lab.llvm.org/buildbot/#/builders/10/builds/6398 Here is the relevant pie

[clang] Enclose code in nondebug in #ifndef NDEBUG/#endif (PR #142189)

2025-05-30 Thread Walter Lee via cfe-commits
https://github.com/googlewalt closed https://github.com/llvm/llvm-project/pull/142189 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] Enclose code in nondebug in #ifndef NDEBUG/#endif (PR #142189)

2025-05-30 Thread via cfe-commits
https://github.com/DeanSturtevant1 created https://github.com/llvm/llvm-project/pull/142189 A previous to InterpBuiltin.cpp fixed an unused variable warning by using [[maybe unused]] and (void). The code actually serves no useful purpose in non-debug builds, so let's not include it there. >F

[clang] Enclose code in nondebug in #ifndef NDEBUG/#endif (PR #142189)

2025-05-30 Thread via cfe-commits
https://github.com/cor3ntin approved this pull request. https://github.com/llvm/llvm-project/pull/142189 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] Enclose code in nondebug in #ifndef NDEBUG/#endif (PR #142189)

2025-05-30 Thread via cfe-commits
https://github.com/DeanSturtevant1 updated https://github.com/llvm/llvm-project/pull/142189 >From 2a78358b97c8a050cb75842094b0b69ece8e49e9 Mon Sep 17 00:00:00 2001 From: DeanSturtevant1 Date: Fri, 30 May 2025 13:41:21 -0400 Subject: [PATCH 1/2] Enclose code that does nothing useful in nondebug

[clang] Enclose code in nondebug in #ifndef NDEBUG/#endif (PR #142189)

2025-05-30 Thread via cfe-commits
github-actions[bot] wrote: :warning: C/C++ code formatter, clang-format found issues in your code. :warning: You can test this locally with the following command: ``bash git-clang-format --diff HEAD~1 HEAD --extensions cpp -- clang/lib/AST/ByteCode/InterpBuiltin.cpp ``

[clang] Enclose code in nondebug in #ifndef NDEBUG/#endif (PR #142189)

2025-05-30 Thread via cfe-commits
llvmbot wrote: @llvm/pr-subscribers-clang Author: None (DeanSturtevant1) Changes A previous change to InterpBuiltin.cpp fixed an unused variable warning by using [[maybe unused]] and (void). The code actually serves no useful purpose in non-debug builds, so let's not include it there.

[clang] Enclose code in nondebug in #ifndef NDEBUG/#endif (PR #142189)

2025-05-30 Thread via cfe-commits
https://github.com/DeanSturtevant1 edited https://github.com/llvm/llvm-project/pull/142189 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits