Blender Git Commit Log

Git Commits -> Revision 2ddecff

Revision 2ddecff by Ankit Meel (master)
October 26, 2020, 09:32 (GMT)
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

Commit Details:

Full Hash: 2ddecfffc3d3a3a1db4ae45e8665caa2a85ab43a
Parent Commit: 4b7abde
Lines Changed: +3, -1

1 Modified Path:

/source/blender/blenkernel/intern/lib_id_delete.c (+3, -1) (Diff)
By: Miika HämäläinenLast update: Nov-07-2014 14:18MiikaHweb | 2003-2021