[PATCH] D44103: Adding additional UNSUPPORTED platform in libcxx/test/std/strings/basic.string/string.cons/iter_alloc_deduction.fail.cpp
Volodymyr Sapsai via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Mar 5 14:45:26 PST 2018
vsapsai added a comment.
In https://reviews.llvm.org/D44103#1027664, @vsk wrote:
> Is the test expected to pass with clang-902? If not, then it'd be best to use "apple-clang-9" instead of "apple-clang-9.0".
That means that most likely it should be `apple-clang-8` instead of 8.0. But on the second thought it might be better to keep .fail.cpp and .pass.cpp more in sync. I think
// XFAIL: libcpp-no-deduction-guides
is better than listing all unsupported clang versions (see r325205 <https://reviews.llvm.org/rL325205>). I didn't test it but if it works, I prefer this approach.
More information about the cfe-commits