diff options
author | Ankit Meel <ankitjmeel@gmail.com> | 2020-10-26 12:32:20 +0300 |
---|---|---|
committer | Ankit Meel <ankitjmeel@gmail.com> | 2020-10-26 12:32:20 +0300 |
commit | 2ddecfffc3d3a3a1db4ae45e8665caa2a85ab43a (patch) | |
tree | 1f48d4ee7187feec35a2bdff1b215c764b6014f6 /source/blender/blenkernel/intern/lattice_deform_test.cc | |
parent | 4b7abde11df9bd6ed759a28286ce3a05147e08c8 (diff) |
Fix T81077 id_management test on macOS
This looks like a optimizer bug where it makes wrong assumptions.
The code inside lib_id_delete:264 on rBafd13710b897cc1c11b
`for (id = last_remapped_id->next; id; id = id->next) {..}`
is not executed in release/relwithdebinfo builds.
This can be "fixed" by several ways:
- Adding a line that prints the `last_remapped_id->name` right before
the said for-loop starts.
- Turning off optimization for the whole function `id_delete`:
`#pragma clang optimize off/on` Ray Molenkamp
- Marking `last_remapped_id` volatile. Julian Eisel
- Marking `tagged_deleted_ids` volatile. But it adds a warning when
calling `BLI_addtail`: discards volatile qualifier. Discovered by
accident.
Fix T81077
Reviewed By: mont29
Maniphest Tasks: T81077
Differential Revision: https://developer.blender.org/D9315
Diffstat (limited to 'source/blender/blenkernel/intern/lattice_deform_test.cc')
0 files changed, 0 insertions, 0 deletions