[all-commits] [llvm/llvm-project] af0ee6: [Clang] Support MSPropertyRefExpr as placement arg...

Sirraide via All-commits all-commits at lists.llvm.org
Wed Jan 17 15:09:43 PST 2024


  Branch: refs/heads/main
  Home:   https://github.com/llvm/llvm-project
  Commit: af0ee617fc5f69051297b0c23f8c818b20f02c3a
      https://github.com/llvm/llvm-project/commit/af0ee617fc5f69051297b0c23f8c818b20f02c3a
  Author: Sirraide <74590115+Sirraide at users.noreply.github.com>
  Date:   2024-01-17 (Wed, 17 Jan 2024)

  Changed paths:
    M clang/docs/ReleaseNotes.rst
    M clang/include/clang/Sema/Sema.h
    M clang/lib/Sema/SemaExpr.cpp
    M clang/lib/Sema/SemaExprCXX.cpp
    A clang/test/CodeGenCXX/ms-property-new.cpp
    A clang/test/CodeGenCXX/placement-new-ms-__noop.cpp
    A clang/test/CodeGenObjCXX/property-placement-new.mm
    M clang/test/SemaCXX/builtin-std-move.cpp
    A clang/test/SemaCXX/ms-property-new.cpp
    A clang/test/SemaCXX/placement-new-bound-member-function.cpp
    A clang/test/SemaCXX/placement-new-builtin.cpp
    A clang/test/SemaCXX/placement-new-matrix.cpp
    A clang/test/SemaCXX/placement-new-ms-__noop.cpp
    A clang/test/SemaObjCXX/property-placement-new.mm

  Log Message:
  -----------
  [Clang] Support MSPropertyRefExpr as placement arg to new-expression (#75883)

It seems we were forgetting to call `checkArgsForPlaceholders` on the
placement arguments of new-expressions in Sema. I don't think that was
intended—at least doing so doesn't seem to break anything—so this pr
adds that.

This also fixes #65053

---------

Co-authored-by: Erich Keane <ekeane at nvidia.com>




More information about the All-commits mailing list