[PATCH] D139429: [clang] Add test for CWG418

Shafik Yaghmour via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Tue Dec 6 10:16:48 PST 2022


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



More information about the cfe-commits mailing list