[PATCH] D130448: [Reassociate][NFC] Use an appropriate `dyn_cast` for `BinaryOperator`

[Sanjay Patel via Phabricator via llvm-commits]

spatel accepted this revision.
spatel added a comment.

In D130448#3675592 <https://reviews.llvm.org/D130448#3675592>, @nikic wrote:

> LG. This should not affect constexpr behavior, because BinaryOperator is, despite its wonderful name, not an Operator (unlike OverflowingBinaryOperator -- go figure!)

Wow...I didn't notice that in all my time working on LLVM. :)


CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D130448/new/

https://reviews.llvm.org/D130448