[PATCH] D127641: [clang-cl][MSVC] Enable /Zc:alignedNew for C++17 and /Zc:sizedDealloc by default

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`.

Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D127641/new/

https://reviews.llvm.org/D127641