[clang] [clang][CFG] Sequence RHS before LHS for overloaded assignment operators in CFG to match C++17 rules (PR #181113)

[Balázs Benics via cfe-commits]

================
@@ -2542,6 +2543,19 @@ CFGBlock *CFGBuilder::VisitChildren(Stmt *S) {
   return B;
 }
 
+CFGBlock *CFGBuilder::VisitCallExprChildren(CallExpr *C) {
+  // C++17 onwards require that the right operand is sequenced before the left
+  // operand.
+  if (auto *OCE = dyn_cast<CXXOperatorCallExpr>(C)) {
----------------
steakhal wrote:

I think I agree with you. I don't see the value of the comment there about C++17.

https://github.com/llvm/llvm-project/pull/181113