[PATCH] D140547: Perform access checking to private members in simple requirement.

Erich Keane via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jan 13 06:19:45 PST 2023


erichkeane added a subscriber: aaron.ballman.
erichkeane added a comment.

In D140547#4050936 <https://reviews.llvm.org/D140547#4050936>, @ilya-biryukov wrote:

> In D140547#4050752 <https://reviews.llvm.org/D140547#4050752>, @uabelho wrote:
>
>> Anyone else see this?
>
> I have not checked, but I would not be surprised if we hit the stack size limits with asan enabled
> @usaxena95, maybe reduce the number of instantiations from `10001` to `1001` or `101`? It should not change the intention of the test and fix this failure.

This is a regression that was noticed by @aaron.ballman even without ASAN, so this is something that needs doing ASAP.  The branch is happening around the 24th, so this needs to be fixed by then.



================
Comment at: clang/lib/Sema/SemaTemplateInstantiate.cpp:1366
       LocalInstantiationScope Scope(SemaRef, /*CombineWithOuterScope=*/true);
-      return inherited::TransformRequiresExpr(E);
+      auto TransReq = inherited::TransformRequiresExpr(E);
+      if (TransReq.isInvalid())
----------------
'auto' isn't allowed here by coding standard, this needs to be ExprResult.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D140547



More information about the cfe-commits mailing list