[PATCH] D50766: Fix false positive unsequenced access and modification warning in array subscript expression.

Richard Smith - zygoloid via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Oct 11 13:23:44 PDT 2018


rsmith added a comment.

In https://reviews.llvm.org/D50766#1262460, @stryku wrote:

> @rsmith Of course. Not sure what's better: to submit one patch per case or submit all of them in one patch (didn't deal much with them, not sure how big will the change be)?


It's really up to you. I don't think a patch that does them all would be too big to review, and doing N small reviews isn't really much more work than one larger review, so please go with whichever approach best suits your workflow.


https://reviews.llvm.org/D50766





More information about the cfe-commits mailing list