[PATCH] D127641: [clang-cl][MSVC] Enable /Zc:alignedNew for C++17 and /Zc:sizedDealloc by default
Hans Wennborg via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Tue Jun 21 03:35:03 PDT 2022
hans added a comment.
In D127641#3597360 <https://reviews.llvm.org/D127641#3597360>, @steplong wrote:
> It looks like misc-new-delete-overloads.cpp is failing on line 20:
> On Line 16, it says sized deallocations are not enabled by default, but this patch turns it on by default for MSVC. Should I unsupport this test for windows or is there a way to pass `/Zc:sizedDealloc-` only for windows?
You could add a target to the test so that it doesn't target windows-msvc, e.g. `-target x86_64-unknown-linux`.
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
More information about the cfe-commits