This is an archive of the discontinued LLVM Phabricator instance.

Adds a warning for aligned new in MSVC before C++17
AbandonedPublic

Authored by luken-google on Jun 13 2022, 12:23 PM.

Details

Reviewers
None
Summary

MSVC requires C++17 to support aligned new and defining the macro
__cpp_aligned_new. Clang does not, which could result in incompatibility
between clang-cl and msvc-built binaries. Issue a warning when this
use case is detected.

Diff Detail

Event Timeline

luken-google created this revision.Jun 13 2022, 12:23 PM
Herald added a project: Restricted Project. · View Herald TranscriptJun 13 2022, 12:23 PM
luken-google requested review of this revision.Jun 13 2022, 12:23 PM
Herald added a project: Restricted Project. · View Herald TranscriptJun 13 2022, 12:23 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript
luken-google abandoned this revision.Jun 14 2022, 8:05 AM

Mistakenly updated change here: https://reviews.llvm.org/D127748