[zeek/spicy] 7cdb08: Merge remote-tracking branch 'origin/dependabot/su...

[Benjamin Bannier]

Branch: refs/heads/topic/bbannier/cxx-struct-parameter-passing
Home: https://github.com/zeek/spicy
Commit: 7cdb089a9e387fa64c1ea4c341d7e30f00c557c2
https://github.com/zeek/spicy/commit/7cdb089a9e387fa64c1ea4c341d7e30f00c557c2
Author: Benjamin Bannier <benjamin...@corelight.com>
Date: 2025-09-30 (Tue, 30 Sep 2025)

Changed paths:
M 3rdparty/utfcpp

Log Message:
-----------
Merge remote-tracking branch 'origin/dependabot/submodules/3rdparty/utfcpp-961e7a3'


Commit: c910097eb10018d7bc3ee7aa8eb6fe28a1343dcd
https://github.com/zeek/spicy/commit/c910097eb10018d7bc3ee7aa8eb6fe28a1343dcd
Author: Benjamin Bannier <benjamin...@corelight.com>
Date: 2025-10-01 (Wed, 01 Oct 2025)

Changed paths:
M hilti/toolchain/include/compiler/detail/cxx/elements.h
M hilti/toolchain/src/compiler/cxx/elements.cc
A tests/hilti/codegen/struct-parameter-passing.hlt

Log Message:
-----------
Fix C++ code generation for struct parameters passed as references.

Previously we could generate C++ moves for constructor arguments passed
as references. This is semantically incorrect, but seems to have had no
ill effects on code we generated. Since it could still confuse readers
and static analysis tools fix this to only generate moves if the
argument was passed as `copy`, but write out copies for all other
argument passing. This more clearly reflects what was already happening.


Compare: https://github.com/zeek/spicy/compare/7cdb089a9e38%5E...c910097eb100

To unsubscribe from these emails, change your notification settings at https://github.com/zeek/spicy/settings/notifications

[Benjamin Bannier]