Homec4science

fixed weird memory bug in the default move constructor of CellBase

Authored by junge on Jun 14 2018, 12:37.

Description

fixed weird memory bug in the default move constructor of CellBase

Summary: turns out that the default move container in combination with clang segfaults under certain (unclear) cicumstances, because the move constructor of the optional appears to be busted in gcc 7.2. Copying it (K) instead of moving it fixes the issue, and since it is a reference, the cost is practically nil.

Test Plan: CI passes

Reviewers: afalsafi

Reviewed By: afalsafi

Differential Revision: https://c4science.ch/D188

Details

Committed
jungeJun 14 2018, 12:37
Pushed
jungeJun 14 2018, 16:27
Reviewer
afalsafi
Differential Revision
Restricted Differential Revision
Parents
rMUSPECTRE8a8455811877: Refactoring field collections, FFT engines and solvers
Branches
Unknown
Tags
Unknown

Event Timeline

Restricted Application completed building Restricted Buildable.Jun 14 2018, 17:24