Skip to content

Conversation

@evedovelli
Copy link
Contributor

Proposed changes

This change fixes the following failure when building with build type Debug:

/projects/composable_kernel/include/ck_tile/ops/flatmm/pipeline/flatmm_pipeline_agmem_bgmem_creg_v1.hpp:62:21: error: left operand of comma operator has no effect [-Werror,-Wunused-value]
   62 |             assert(("Wrong TailNumber!", false));
      |                     ^~~~~~~~~~~~~~~~~~~
/usr/include/assert.h:103:27: note: expanded from macro 'assert'
  103 |      (static_cast <bool> (expr)                                         \
      |                           ^~~~

Checklist

  • I have added tests relevant to the introduced functionality, and the unit tests are passing locally
  • I have added the test to REGRESSION_TESTS list defined at the top of CMakeLists.txt in tests/CMakeLists.txt, IF the test takes more than 30 seconds to run.
  • I have added inline documentation which enables the maintainers with understanding the motivation
  • I have removed the stale documentation which is no longer relevant after this pull request
  • (If this change is user-facing) I have added release notes which provide the end users with a brief summary of the improvement from this pull request
  • I have run clang-format on all changed files
  • Any dependent changes have been merged

@evedovelli evedovelli merged commit e302079 into develop Jan 15, 2026
21 checks passed
@evedovelli evedovelli deleted the users/evedovel/fix-debug-build branch January 15, 2026 14:35
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

bug Something isn't working compiler bug urgency_low

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants