[PATCH] D57747: [Sema] SequenceChecker: Fix handling of operator ||, && and ?:

Bruno Ricci via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Dec 12 05:03:34 PST 2019


riccibruno added a comment.

In D57747#1774848 <https://reviews.llvm.org/D57747#1774848>, @xbolva00 wrote:

> Does the whole stack of patch need to be commited at once or maybe you can land them individually?


Hi, thanks for looking at this patch series !

If I remember correctly (it has been a while now) these patches could be landed independently. I was sort of hoping that D57660 <https://reviews.llvm.org/D57660> would get accepted and that I could then land the other patches without rebasing them (the tests are especially annoying to rebase). That said if D57660 <https://reviews.llvm.org/D57660> is a problem then I can totally do the work and land the other already accepted patches.


Repository:
  rC Clang

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

https://reviews.llvm.org/D57747





More information about the cfe-commits mailing list