[zeek/spicy] f9346d: Merge branch 'topic/bbannier/gh-windows-ci'

[Benjamin Bannier]

Branch: refs/heads/topic/bbannier/issue-2404
Home: https://github.com/zeek/spicy
Commit: f9346d46147310e3ea2f97aba8d671cca4cfa183
https://github.com/zeek/spicy/commit/f9346d46147310e3ea2f97aba8d671cca4cfa183
Author: Benjamin Bannier <benjamin...@corelight.com>
Date: 2026-06-16 (Tue, 16 Jun 2026)

Changed paths:
M .github/workflows/check.yml
M 3rdparty/libunistd
M CHANGES
M VERSION
M ci/windows/build.cmd
M ci/windows/prepare.cmd
M ci/windows/test.cmd

Log Message:
-----------
Merge branch 'topic/bbannier/gh-windows-ci'


Commit: d9ecb209f2593b038c10ea6d8d05e4a96873d2e9
https://github.com/zeek/spicy/commit/d9ecb209f2593b038c10ea6d8d05e4a96873d2e9
Author: Benjamin Bannier <benjamin...@corelight.com>
Date: 2026-06-16 (Tue, 16 Jun 2026)

Changed paths:
M hilti/toolchain/src/compiler/cfg.cc

Log Message:
-----------
Include coercions in detection of expressions with side effects

We previously would always consider expressions with coercions as having
side effects which in at least #2404 lead to us generating code which
caused a most vexing parse. Since coercions do not have side effects
themself, extend our detection to include them by querying the behavior
of the coerced expression. This should fix the majority of new failures
introduced by recent rewrites which removed the LHS of assignments; we
will do more structural fixes in follow-up patches.

Closes #2404.


Commit: 63cb5a27cd09c2f955cfebb7616cbb4d930f117f
https://github.com/zeek/spicy/commit/63cb5a27cd09c2f955cfebb7616cbb4d930f117f
Author: Benjamin Bannier <benjamin...@corelight.com>
Date: 2026-06-16 (Tue, 16 Jun 2026)

Changed paths:
M hilti/toolchain/src/compiler/codegen/coercions.cc
M hilti/toolchain/src/compiler/codegen/ctors.cc
M tests/Baseline/hilti.hiltic.print.globals/output
M tests/Baseline/hilti.hiltic.print.globals/output2
M tests/Baseline/hilti.hiltic.print.hello-world/output
M tests/Baseline/hilti.hiltic.print.import/output
M tests/Baseline/hilti.hiltic.print.import/output2
M tests/Baseline/hilti.hiltic.print.yield/output
M tests/Baseline/hilti.types.tuple.coercion/output
M tests/Baseline/spicy.tools.preprocessor/output
M tests/Baseline/spicy.tools.spicyc-hello-world/test.hlt

Log Message:
-----------
Disambiguate constructor calls from function declarations

We previously called constructors as `T(..)` which without context could
lead to most vexing parses. In this patch we replace most constructor
calls in type creations and coercions with `T{..}` instead to avoid any
downstream issues, especially since we have passes which might remove
the LHS.


Commit: 6fd4e5bfa93f91a193533bd6aff8787ed34c2eb2
https://github.com/zeek/spicy/commit/6fd4e5bfa93f91a193533bd6aff8787ed34c2eb2
Author: Benjamin Bannier <benjamin...@corelight.com>
Date: 2026-06-16 (Tue, 16 Jun 2026)

Changed paths:
M hilti/toolchain/src/compiler/codegen/coercions.cc

Log Message:
-----------
Stop emitting unneeded constructor in codegen for list comprehensions

Since `Set` can implicitly construct from an initializer list, there is
no need to add an explicit constructor call, so in this patch we remove
it to further reduced potential for us accidentially generating code
causing most vexing parses.


Compare: https://github.com/zeek/spicy/compare/44b2f4efcbc0...6fd4e5bfa93f

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