[PATCH] D103252: [C++4OpenCL] Fix missing address space on implicit move assignment operator
Abhinav Gaba via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Thu Jun 10 10:29:04 PDT 2021
abhinavgaba added inline comments.
================
Comment at: clang/test/AST/ast-dump-implicit-members.clcpp:10
+
+// CHECK: CXXConstructorDecl {{.*}} implicit used constexpr S 'void () __generic noexcept'
+// CHECK: CXXConstructorDecl {{.*}} implicit constexpr S 'void (const __generic S &) __generic'
----------------
olestrohm wrote:
> abhinavgaba wrote:
> > Could you please relax these checks? The test fails with `-triple i386-pc-win32` with errors like this:
> > ```
> > ast-dump-implicit-members.clcpp:10:11: error: CHECK: expected string not found in input
> > // CHECK: CXXConstructorDecl {{.*}} implicit used constexpr S 'void () __generic noexcept'
> > ^
> > <stdin>:1:1: note: scanning from here
> > Dumping __NSConstantString:
> > ^
> > <stdin>:16:18: note: possible intended match here
> > |-CXXConstructorDecl 0x10b6c718 <col:8> col:8 implicit used constexpr S 'void () __attribute__((thiscall)) __generic noexcept' inline default trivial
> > ^
> > ```
> Hi, sorry for this.
> I have relaxed the tests in rGac677e69bdfc, so hopefully that should fix it.
Thanks for the fix, @olestrohm.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D103252/new/
https://reviews.llvm.org/D103252
More information about the cfe-commits
mailing list