Closed vgvassilev closed 2 weeks ago
clang-tidy review says "All clean, LGTM! :+1:"
All modified and coverable lines are covered by tests :white_check_mark:
Project coverage is 93.79%. Comparing base (
8bc762c
) to head (e433bc9
).
clang-tidy review says "All clean, LGTM! :+1:"
clang-tidy review says "All clean, LGTM! :+1:"
We don't need the enableTBR
field in the RMV at all, right? I suggest we remove it from ReverseModeVisitor.h as well. Other than that, the PR looks good to me.
We don't need the
enableTBR
field in the RMV at all, right? I suggest we remove it from ReverseModeVisitor.h as well. Other than that, the PR looks good to me.
Good point, I screwed up with the rebase. Now it should be better.
clang-tidy review says "All clean, LGTM! :+1:"
LGTM!
Partially addresses #721.