[zeek/spicy] 143240: Extend printer to always output functions' calling...

[Robin Sommer]

Branch: refs/heads/topic/robin/misc-opt
Home: https://github.com/zeek/spicy
Commit: 14324017b7d3d219e2f6e472bc6c2a23c41e3671
https://github.com/zeek/spicy/commit/14324017b7d3d219e2f6e472bc6c2a23c41e3671
Author: Robin Sommer <ro...@corelight.com>
Date: 2026-02-25 (Wed, 25 Feb 2026)

Changed paths:
M hilti/toolchain/src/compiler/printer.cc
M tests/Baseline/spicy.optimization.default-parser-functions/log
M tests/Baseline/spicy.optimization.default-parser-functions/noopt.hlt
M tests/Baseline/spicy.optimization.default-parser-functions/opt.hlt
M tests/Baseline/spicy.optimization.feature_requirements/log
M tests/Baseline/spicy.optimization.feature_requirements/noopt.hlt
M tests/Baseline/spicy.optimization.feature_requirements/opt.hlt
M tests/Baseline/spicy.optimization.unused-functions/log
M tests/Baseline/spicy.optimization.unused-functions/noopt.hlt
M tests/Baseline/spicy.optimization.unused-functions/opt.hlt
M tests/Baseline/spicy.optimization.unused-types/log
M tests/Baseline/spicy.optimization.unused-types/noopt.hlt
M tests/Baseline/spicy.optimization.unused-types/opt.hlt

Log Message:
-----------
Extend printer to always output functions' calling conventions.

This was missing for some function types, so that one wasn't able to
see what their calling convention was.


Commit: 0b032e1e7f96ff5baeca41badd4438b93f217a37
https://github.com/zeek/spicy/commit/0b032e1e7f96ff5baeca41badd4438b93f217a37
Author: Robin Sommer <ro...@corelight.com>
Date: 2026-02-25 (Wed, 25 Feb 2026)

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

Log Message:
-----------
Codegen fix for wrong qualification of non-extern struct members.


Commit: 21a1532e864ac448299f5cfe7814adc6c6cad99f
https://github.com/zeek/spicy/commit/21a1532e864ac448299f5cfe7814adc6c6cad99f
Author: Robin Sommer <ro...@corelight.com>
Date: 2026-02-25 (Wed, 25 Feb 2026)

Changed paths:
M hilti/toolchain/include/compiler/detail/optimizer/optimizer.h
M hilti/toolchain/include/compiler/detail/optimizer/pass.h
M hilti/toolchain/src/compiler/optimizer/passes/remove-unused-fields.cc

Log Message:
-----------
Add optimizer helpers to determine if AST entities can be modified.

These honor ``--no-strict-public-api` as well as various attributes
and other AST properties.

Also port the `remove-unused-fields` pass over to use one of them,
more to come.


Commit: 4b4eaf8c7ffc5c1b5be24cb951a774501e8f9fdd
https://github.com/zeek/spicy/commit/4b4eaf8c7ffc5c1b5be24cb951a774501e8f9fdd
Author: Robin Sommer <ro...@corelight.com>
Date: 2026-02-25 (Wed, 25 Feb 2026)

Changed paths:
M hilti/toolchain/src/compiler/optimizer/passes/remove-unused-params.cc
M tests/Baseline/hilti.optimization.unreachable-code/log
M tests/Baseline/hilti.optimization.unreachable-code/opt.hlt
M tests/Baseline/hilti.optimization.unused-struct-fields/output
M tests/Baseline/spicy.optimization.default-parser-functions/log
M tests/Baseline/spicy.optimization.default-parser-functions/opt.hlt
M tests/Baseline/spicy.optimization.feature_requirements/log
M tests/Baseline/spicy.optimization.feature_requirements/opt.hlt
M tests/Baseline/spicy.optimization.unused-functions/log
M tests/Baseline/spicy.optimization.unused-functions/opt.hlt
M tests/Baseline/spicy.optimization.unused-types/log
M tests/Baseline/spicy.optimization.unused-types/opt.hlt
M tests/hilti/output/optimization/remove-method-params.hlt
M tests/spicy/optimization/error-push-pop.spicy
M tests/spicy/optimization/offset_vs_random-access.spicy
M tests/spicy/types/regexp/parse-ctor-captures.spicy
M tests/spicy/types/unit/external-hook.spicy
M tests/spicy/types/unit/switch-shared-id.spicy

Log Message:
-----------
Let the `remove-unused-params` pass honor `--no-strict-public-api`.


Commit: 523ad4abf359bd4ecd06d45d789ac10d6be5ddf4
https://github.com/zeek/spicy/commit/523ad4abf359bd4ecd06d45d789ac10d6be5ddf4
Author: Robin Sommer <ro...@corelight.com>
Date: 2026-02-25 (Wed, 25 Feb 2026)

Changed paths:
M hilti/toolchain/src/compiler/optimizer/passes/propagate-function-returns.cc
M tests/hilti/output/optimization/no-propagate-returns.hlt

Log Message:
-----------
Let the `propagate-function-returns` pass honor `--no-strict-public-api`.


Commit: e7e6e9c24d8a0767192d77d22eeba0d96eadafa9
https://github.com/zeek/spicy/commit/e7e6e9c24d8a0767192d77d22eeba0d96eadafa9
Author: Robin Sommer <ro...@corelight.com>
Date: 2026-02-25 (Wed, 25 Feb 2026)

Changed paths:
M hilti/toolchain/src/compiler/optimizer/passes/dead-code-static.cc
M tests/Baseline/hilti.optimization.peephole-default-void/opt.hlt
M tests/Baseline/hilti.optimization.peephole-try-catch/opt.hlt
M tests/Baseline/hilti.optimization.unreachable-code/log
M tests/Baseline/hilti.optimization.unreachable-code/opt.hlt
A tests/Baseline/hilti.optimization.unused-methods/output
M tests/Baseline/hilti.optimization.unused-struct-fields/output
M tests/Baseline/hilti.optimization.unused_function/log
M tests/Baseline/hilti.optimization.unused_function/opt.hlt
M tests/Baseline/spicy.optimization.default-parser-functions/log
M tests/Baseline/spicy.optimization.feature_requirements/log
M tests/Baseline/spicy.optimization.unused-functions/log
M tests/Baseline/spicy.optimization.unused-functions/opt.hlt
M tests/Baseline/spicy.optimization.unused-types/log
M tests/Baseline/spicy.optimization.unused-types/opt.hlt
M tests/hilti/codegen/struct-parameter-passing.hlt
M tests/hilti/hiltic/cc/enum-prototype.hlt
M tests/hilti/optimization/flatten-block.hlt
A tests/hilti/optimization/unused-methods.hlt
M tests/hilti/types/struct/static-method-extern.hlt

Log Message:
-----------
Let the `dead-code-static` pass honor `--no-strict-public-api`.

There were also a couple of places either not quite working as
intended or not (any longer) needed, so tweaking things a bit. Also
adding one extension for removing not needed member functions.


Compare: https://github.com/zeek/spicy/compare/47f6c50ff5b3...e7e6e9c24d8a

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