[clang] [Clang] Support MSPropertyRefExpr as placement arg to new-expression (PR #75883)

via cfe-commits cfe-commits at lists.llvm.org
Wed Jan 3 14:25:56 PST 2024


================
@@ -2286,6 +2286,9 @@ ExprResult Sema::BuildCXXNew(SourceRange Range, bool UseGlobal,
   bool PassAlignment = getLangOpts().AlignedAllocation &&
                        Alignment > NewAlignment;
 
+  if (CheckArgsForPlaceholders(PlacementArgs))
----------------
Sirraide wrote:

I’ll take a look at what other expressions are actually affected by this other than `MSPropertyRefExpr`s. (or do you mean we should just add more placement-new tests in general? Because I would assume we already have a bunch of those, but I haven’t checked to be fair?)

https://github.com/llvm/llvm-project/pull/75883


More information about the cfe-commits mailing list