[clang] [Clang] Support MSPropertyRefExpr as placement arg to new-expression (PR #75883)
    via cfe-commits 
    cfe-commits at lists.llvm.org
       
    Wed Jan  3 15:17:31 PST 2024
    
    
  
================
@@ -2286,6 +2286,9 @@ ExprResult Sema::BuildCXXNew(SourceRange Range, bool UseGlobal,
   bool PassAlignment = getLangOpts().AlignedAllocation &&
                        Alignment > NewAlignment;
 
+  if (CheckArgsForPlaceholders(PlacementArgs))
----------------
Sirraide wrote:
Makes sense. It seems like this affects only a small number of builtin types, so in theory, adding tests for every expression that could possibly have those types ought to do (if we don’t already have tests for those); I’m currently collecting a list of everything that could be affected by this.
https://github.com/llvm/llvm-project/pull/75883
    
    
More information about the cfe-commits
mailing list