Closed AdrianSchmidt1 closed 1 week ago
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 65.95%. Comparing base (
fbd8948
) to head (6c58e40
). Report is 8 commits behind head on master.
:umbrella: View full report in Codecov by Sentry.
:loudspeaker: Have feedback on the report? Share it here.
🚨 Try these New Features:
This PR includes two changes:
double *signature
in the functionzos_pl_forward
inuni5_for.c
which caused problems due to allocation but missing deallocation of memory.freeTapeResources
intaping.c
which has previously been missing. This caused memory leaks when reusing tape IDs even when using the functionremoveTape
before creating the new tape.At a future point it would be very useful to add a note to the
README.md
which points out that the functionremoveTape
must be called before rewriting a tape reusing a tape ID in one execution. This workaround will prevent the aforementioned memory leak until ADOL-C adds the feature to automatically remove the tape before retaping in case the tape ID is reused. (Lot's of 'tape' in one sentence, hope it's clear what I mean)