[PATCH] D121748: [clang][Sema] Better support for ObjC++ in Sema::LookupName

[Alex Langford via Phabricator via cfe-commits]

bulbazord added a comment.

In D121748#3397715 <https://reviews.llvm.org/D121748#3397715>, @arphaman wrote:

> I don't think this patch is sound. I found this problem with this change when the following file is compiled in Objective-C++ mode:
>
>   template <class T, bool umax = true>
>   T umax(T a, T b) {
>     return a;
>   }
>
> produces this error:
>
>   test.mm:3:3: error: declaration of 'umax' shadows template parameter
>   T umax(T a, T b) {
>     ^
>   test.mm:2:25: note: template parameter is declared here
>   template <class T, bool umax = true>
>                           ^
>
> which is unexpected and isn't produced in C++ mode.

Could you tell me what flags you gave to clang to compile the test? I'd like to test a few things.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D121748