Bytecode Interpreter: Migrate away from `CheckEvaluationResult`

[tbaederr]

We're currently still calling the CheckEvaluationResult from ExprConstant.cpp, but we already have similar checks in EvaluationResult etc. that we should use instead. This should also be more efficient since we can do the checks before creating the APValue for the returned value.

[llvmbot]

@llvm/issue-subscribers-clang-frontend

Author: Timm Baeder (tbaederr)

We're currently still calling the `CheckEvaluationResult` from `ExprConstant.cpp`, but we already have similar checks in `EvaluationResult` etc. that we should use instead. This should also be more efficient since we can do the checks _before_ creating the `APValue` for the returned value.