shafik added a subscriber: aaron.ballman.
shafik added a comment.

In D139429#3975028 <https://reviews.llvm.org/D139429#3975028>, @Endill wrote:

> I opened #59363 <https://github.com/llvm/llvm-project/issues/59363> on bug 
> tracker.
> What should I do about this patch then? I believe availability `no` should 
> come with a failing test. Should I include example from [over.match.best]/4 
> as another example in the test?

Was chatting with @aaron.ballman and he recommends adding a `FIXME` comment 
along the lines of:

  // FIXME: this shouldn't pass, it should fail!


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D139429

_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to