Oliver Stöneberg
Oliver Stöneberg
> There should probably be a check for `value_size < 8` in `truncateIntValue()`, given that `bigint` is only 64 bits. That check should be dependent on the actual size of...
> CI is now failing due to errors not related to this PR. Yes, there was a bad merge. It rarely happens. Please rebase.
> I do not really understand how the CI error in undefined behaviour sanitizer build has anything to do with my changes. That is the selfcheck/bootstrap/dogfood. We run our code...
This requires https://github.com/danmar/simplecpp/pull/347 to merged and sync'd down first.
I will try to put the refactoring which does not require the simplecpp changes into a separate PR.
The callgrind CI job shows a small reduction in Ir: `64,070,509,276` -> `63,862,736,388`.
@danmar This requires a simplecpp release.
This exposed a false negative: https://trac.cppcheck.net/ticket/12711.
@danmar This still requires a simplecpp release.
> some kind of small benchmark that measures actual time would be interesting. I will provide additional values. This will also help with value evaluation stuff which is not executed...