[zeek/spicy] d4825d: fixup! DO NOT MERGE: Simplify CI for testing

0 views
Skip to first unread message

Benjamin Bannier

unread,
Jun 3, 2026, 3:26:50 AM (3 days ago) Jun 3
to spicy-...@zeek.org
Branch: refs/heads/topic/bbannier/msvc-warnings
Home: https://github.com/zeek/spicy
Commit: d4825d9ca6414242c4e433cf3e7d4f507775ca42
https://github.com/zeek/spicy/commit/d4825d9ca6414242c4e433cf3e7d4f507775ca42
Author: Benjamin Bannier <benjamin...@corelight.com>
Date: 2026-06-03 (Wed, 03 Jun 2026)

Changed paths:
R .github/workflows/benchmark.yml

Log Message:
-----------
fixup! DO NOT MERGE: Simplify CI for testing


Commit: 475826888c22cad1dd171e52caec3bcae9a322bd
https://github.com/zeek/spicy/commit/475826888c22cad1dd171e52caec3bcae9a322bd
Author: Benjamin Bannier <benjamin...@corelight.com>
Date: 2026-06-03 (Wed, 03 Jun 2026)

Changed paths:
M hilti/toolchain/include/compiler/detail/cfg.h

Log Message:
-----------
Hide GCC pragma from other compilers


Commit: 50e65b5e27198ca94333f16cea01b4cf7b2d449e
https://github.com/zeek/spicy/commit/50e65b5e27198ca94333f16cea01b4cf7b2d449e
Author: Benjamin Bannier <benjamin...@corelight.com>
Date: 2026-06-03 (Wed, 03 Jun 2026)

Changed paths:
M hilti/runtime/include/logging.h
M hilti/runtime/include/macros.h
M hilti/runtime/include/safe-int.h
M hilti/runtime/include/types/optional.h
M hilti/runtime/include/types/reference.h
M hilti/runtime/include/types/struct.h
M hilti/runtime/include/types/tuple.h
M hilti/runtime/include/util.h
M hilti/runtime/src/types/optional.cc
M hilti/runtime/src/types/tuple.cc
M hilti/toolchain/include/base/logger.h
M hilti/toolchain/include/base/util.h

Log Message:
-----------
Consistently use `[[noreturn]]` in favor of compiler-specific attributes

This also addresses the issue of a couple of functions not being marked
noreturn for msvc which caused it to emit warnings about unhandled code
paths.


Compare: https://github.com/zeek/spicy/compare/1be8b5783f7d...50e65b5e2719

To unsubscribe from these emails, change your notification settings at https://github.com/zeek/spicy/settings/notifications
Reply all
Reply to author
Forward
0 new messages