What made i = i++ + 1; legal in C++17?
In C++11 the act of "assignment", i.e. the side-effect of modifying the LHS, is sequenced after the value computation of the right operand. Note that this is a relatively "weak" guarantee: it produces sequencing only with relation to value computation of the RHS. It says nothing about the side-effects that might be present in the RHS, since occurrence of side-effects is not part of value computation. The requirements of C++11 establish no relative sequencing between the act of assignment and any side-effects of the RHS. This is what creates the potential for UB.
The only hope in this case is any additional guarantees made by specific operators used in RHS. If the RHS used a prefix ++
, sequencing properties specific to the prefix form of ++
would have saved the day in this example. But postfix ++
is a different story: it does not make such guarantees. In C++11 the side-effects of =
and postfix ++
end up unsequenced with relation to each other in this example. And that is UB.
In C++17 an extra sentence is added to the specification of assignment operator:
The right operand is sequenced before the left operand.
In combination with the above it makes for a very strong guarantee. It sequences everything that happens in the RHS (including any side-effects) before everything that happens in the LHS. Since the actual assignment is sequenced after LHS (and RHS), that extra sequencing completely isolates the act of assignment from any side-effects present in RHS. This stronger sequencing is what eliminates the above UB.
(Updated to take into account @John Bollinger's comments.)
You identified the new sentence
The right operand is sequenced before the left operand.
and you correctly identified that the evaluation of the left operand as an lvalue is irrelevant. However, sequenced before is specified to be a transitive relation. The complete right operand (including the post-increment) is therefore also sequenced before the assignment. In C++11, only the value computation of the right operand was sequenced before the assignment.