Blender Git Loki

Fix T89736: Cycles error with persistent data, displacement and motion blur

adaptive_cloth: Mesh: collapse_edge_triangulate() version 2

Fix T89396: Cycles missing passes with multiple view layers and persistent data

Cycles X: Reduce memory usage of OptiX denoiser and CPU render

Allow to modify a temporary buffer created in the DeviceDenoiser.

Moved denoised pass copy to a more accessible function. It is not
very flexible yet, and performance might be improved, but it fits
current usage and is only used from a much more compute-heavy steps
of the render.

Differential Revision: https://developer.blender.org/D11874

Also apply flipping correction when bone's X-axis is aligned with Y.

Cleanup: Clang format

Sorry, missed this in 53cf8e83b37d.

Fix T89775: geometry nodes logging crash during render

Under some circumstances (e.g. when rendering) the geometry
nodes logger is not used. This was missing a simple null check.

Cycles X: Include rebalance time into full report

Useful for investigating various balance strategies.

Differential Revision: https://developer.blender.org/D11887

Cycles X: Improve average time report in the log

Don't count non-finial resolution divider in the average as it is not
accurate.
The wall time is still counted.

Blenlib: Add BLI_assert_msg() for printing an extra string if the assert fails

It always bothered me that we'd do the `BLI_assert(... || !"message")` trick to
print a message alongside the assert, while it should be trivial to have a way
to pass an extra string as additional argument.

This adds `BLI_assert_msg()` with a second argument for a message. E.g.:
```
BLI_assert_msg(
params->rename_id == NULL,
"File rename handling should immediately clear rename_id when done, because otherwise it will keep taking precedence over renamefile.");
```

On failure this will print like this:
```
0 Blender 0x00000001140647a3 BLI_system_backtrace + 291
[...]
13 Blender 0x00000001092647a6 main + 3814
14 libdyld.dylib 0x00007fff203d8f5d start + 1
BLI_assert failed: source/blender/editors/space_file/file_ops.c:2352, file_directory_new_exec(), at 'params->rename_id == ((void*)0)'
File rename handling should immediately clear rename_id when done, because otherwise it will keep taking precedence over renamefile.
```

Reviewed by: Sybren St�vel, Jacques Lucke, Sergey Sharybin, Campbell Barton

Differential Revision: https://developer.blender.org/D11827

Merge branch 'master' into cycles-x

Fix T89765: boolean modifier collection refcount issue

The 'collection' property is flagged PROP_ID_REFCOUNT, so the
modifiers foreachIDLink functions should walk with IDWALK_CB_USER
(instead of IDWALK_CB_NOP).

Otherwise the modifier wont be included as a user for the collection
(e.g. on file read); removing the collection from the modifier will
decrement usercount though (which in worst case scenario makes the
collection orphan and will result in data loss)

Maniphest Tasks: T89765

Differential Revision: https://developer.blender.org/D11877

Make gcc happy

Fix bug in previous commit; customdata_bmesh_set_default now ignores
CD_MESH_ID layers

BM_mesh_create will now add mesh_id customdata layers when asked,
instead of leaving that to the client.

Also semi-fixed uninitialized memory bug in bmesh unit test (dunno
how best to memset a C struct in C++ won't won't run afoul of some
random compiler somewhere).

Let's try that again

Weird linux compile error

Fix more dyntopo sculpt undo memory corruption

Merge branch 'nv-usd-importer' of https://gitlab-master.nvidia.com/omniverse/blender into omniverse

Merge branch 'usd-importer-T81257-merge' of https://gitlab-master.nvidia.com/omniverse/blender into nv-usd-importer