[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