This is an archive of the discontinued LLVM Phabricator instance.

[libcxx][test] Disable bad unique_ptr<T[]> to shared_ptr<U[]> conversion test cases
ClosedPublic

Authored by CaseyCarter on Jan 23 2022, 10:33 AM.

Details

Summary

for non-libc++. I've reported allowance of these conversions as a bug at https://llvm.org/PR53368.

Diff Detail

Event Timeline

CaseyCarter requested review of this revision.Jan 23 2022, 10:33 AM
CaseyCarter created this revision.
Herald added 1 blocking reviewer(s): Restricted Project. · View Herald TranscriptJan 23 2022, 10:33 AM
CaseyCarter edited the summary of this revision. (Show Details)Jan 23 2022, 10:45 AM
Mordante accepted this revision as: Mordante.Jan 24 2022, 10:44 AM
Mordante added a subscriber: Mordante.

LGTM!

libcxx/test/std/utilities/memory/util.smartptr/util.smartptr.shared/util.smartptr.shared.assign/unique_ptr_Y.pass.cpp
129

Thanks for filing a bug report! Can you add a link to the bug in the comments?

Quuxplusone added inline comments.
libcxx/test/std/utilities/memory/util.smartptr/util.smartptr.shared/util.smartptr.shared.assign/unique_ptr_Y.pass.cpp
129

+1, please add // https://llvm.org/PR53368 on each of these lines (and then the longer comment is probably superfluous). Whoever fixes PR53368 should git grep for mentions of it, which will lead them here.

CaseyCarter edited the summary of this revision. (Show Details)

Replace flowery comment with problem report link.

CaseyCarter marked 2 inline comments as done.Jan 24 2022, 10:56 AM
ldionne accepted this revision.Feb 8 2022, 2:40 PM
This revision is now accepted and ready to land.Feb 8 2022, 2:40 PM