code is slightly cleaned up if (captureOrPromotion && !givesCheck) is squashed into if (!givesCheck) - since if it doesn't give check it's already captureOrPromotion according to previous if / else condition

I personnaly would refactor the code by moving
CapturePieceToHistory& captureHistory = thisThread->captureHistory;
at the beginning of the search function and replace the occurances of captureHistory in line 902 and 957. So capture history is easier to use at further places in search.
And then use following:

            if (  !givesCheck
                && lmrDepth < 1 
                && captureHistory[movedPiece][to_sq(move)][type_of(pos.piece_on(to_sq(move)))] < 0)
                      continue;

But that is a matter of taste!

Well yes it will be shorter and easier to read.
The only reason why I did it this in a way it is in pr is to have patch in one place for easier tweaking, it's not close to be optimal in terms of style :)

Thanks, added with cleanups.