https://github.com/sdkrystian created
https://github.com/llvm/llvm-project/pull/91620
This reverts commit 62b5b61f436add042d8729dc9837d055613180d9 (#91498).
>From 7b2f3da17dfc93a4f0aa69ad4da90707b6f2e8b6 Mon Sep 17 00:00:00 2001
From: Krystian Stasiowski
Date: Thu, 9 May 2024 12:30:28 -0400
Su
llvmbot wrote:
@llvm/pr-subscribers-clang
Author: Krystian Stasiowski (sdkrystian)
Changes
This reverts commit 62b5b61f436add042d8729dc9837d055613180d9 (#91498).
---
Full diff: https://github.com/llvm/llvm-project/pull/91620.diff
3 Files Affected:
- (modified) clang/lib/Sema/SemaLookup
https://github.com/erichkeane approved this pull request.
This reverts the problematic parts of the previous patch too, right? If so,
LGTM.
https://github.com/llvm/llvm-project/pull/91620
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
https:
github-actions[bot] wrote:
:warning: C/C++ code formatter, clang-format found issues in your code.
:warning:
You can test this locally with the following command:
``bash
git-clang-format --diff 1494d8849fa2aef575dabd431b0060639f4a57c1
7b2f3da17dfc93a4f0aa69ad4da90707b6f2e8b6 --
sdkrystian wrote:
Is it alright to partially revert commits in a single PR? If so, I'm just going
to revert all the `operator=` related changes from #91498, #90999, and #90152.
https://github.com/llvm/llvm-project/pull/91620
___
cfe-commits mailing l
aeubanks wrote:
should be fine to revert as much as you want in a single PR, just make sure to
mention what you're reverting in the description
https://github.com/llvm/llvm-project/pull/91620
___
cfe-commits mailing list
cfe-commits@lists.llvm.org
htt
erichkeane wrote:
> should be fine to revert as much as you want in a single PR, just make sure
> to mention what you're reverting in the description
Agreed, a revert doesnt have to be a single commit, or even a whole one. Just
enough to get us back to a 'working' state is fine.
Though if yo
https://github.com/sdkrystian updated
https://github.com/llvm/llvm-project/pull/91620
>From 7b2f3da17dfc93a4f0aa69ad4da90707b6f2e8b6 Mon Sep 17 00:00:00 2001
From: Krystian Stasiowski
Date: Thu, 9 May 2024 12:30:28 -0400
Subject: [PATCH 1/4] Revert "[Clang][Sema] Fix lookup of dependent operato
https://github.com/sdkrystian updated
https://github.com/llvm/llvm-project/pull/91620
>From 7b2f3da17dfc93a4f0aa69ad4da90707b6f2e8b6 Mon Sep 17 00:00:00 2001
From: Krystian Stasiowski
Date: Thu, 9 May 2024 12:30:28 -0400
Subject: [PATCH 1/3] Revert "[Clang][Sema] Fix lookup of dependent operato