The new `mask_in_place` variables cover not only valhalla bits but other bits as well and the `mask_in_place` is not a shifted version of the `mask`. This might end up causing confusing. I propose we have a clear pattern and naming scheme here. (As it is in mainline)

Where we first have a section for the bit counts. Followed by bit shifts. Followed by `mask` and `mask_in_place` where the `mask_in_place` is a shifted version of the `mask`.

Then we can have derived masks and specific patterns.