[PATCH] D123182: [Concepts] Fix overload resolution bug with constrained candidates

Nick Kreeger via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Apr 25 07:18:10 PDT 2022


nkreeger added a comment.

In D123182#3471789 <https://reviews.llvm.org/D123182#3471789>, @erichkeane wrote:

> In D123182#3471687 <https://reviews.llvm.org/D123182#3471687>, @nkreeger wrote:
>
>> In D123182#3471661 <https://reviews.llvm.org/D123182#3471661>, @erichkeane wrote:
>>
>>> @nkreeger : Can you please explain your revert, both in the revert commit message (next time), as well as the patch so that the author/rest of us have SOME hint as to why it was reverted?  Frequent reverts make it painful as a downstream, and confusing as a reviewer/implementer as to the state of things.
>>
>> Apologies - I was reverting my commit and accidently had the wrong git-revert commit in my tree locally. I thought I had it cleaned up, but was wrong when I pushed. I ensured the tree was back in place with another revert. Please let me know if this is not the case. Sorry again :-(
>
> I see that now, thanks for the explanation! IN the future, even when reverting your OWN commit, it is appreciated if you explain the WHY to the revert so that it has some level of context.  Particularly when you show some sort of example of the failure (like a link to the broken bot, etc), as it helps the downstream test-failure-analysis a ton.

Roger noted - apologies again I felt horrible!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D123182



More information about the cfe-commits mailing list