[PATCH] D112921: [clang] Enable sized deallocation by default in C++14 onwards

Reid Kleckner via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 1 10:26:14 PDT 2021


rnk added a comment.

Please add a note to clang/docs/ReleaseNotes.rst about the behavior change.

The clangd test failure seems related to this change, and the other ones could be as well.



================
Comment at: clang/lib/Driver/ToolChains/Clang.cpp:6405
+  // by default.
+  if (Arg *A = Args.getLastArg(options::OPT_fsized_deallocation,
+                               options::OPT_fno_sized_deallocation)) {
----------------
Can this be `Args.AddLastArg(CmdArgs, ...);`?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D112921



More information about the cfe-commits mailing list