Skip to content

Conversation

@kazutakahirata
Copy link
Contributor

We've switched to LineLocation from FieldsAre, so we don't need this
"using" directive anymore.

We've switched to LineLocation from FieldsAre, so we don't need this
"using" directive anymore.
@llvmbot
Copy link
Member

llvmbot commented Nov 20, 2024

@llvm/pr-subscribers-llvm-transforms

Author: Kazu Hirata (kazutakahirata)

Changes

We've switched to LineLocation from FieldsAre, so we don't need this
"using" directive anymore.


Full diff: https://github.com/llvm/llvm-project/pull/117004.diff

1 Files Affected:

  • (modified) llvm/unittests/Transforms/Instrumentation/MemProfUseTest.cpp (-1)
diff --git a/llvm/unittests/Transforms/Instrumentation/MemProfUseTest.cpp b/llvm/unittests/Transforms/Instrumentation/MemProfUseTest.cpp
index fb87d649a249cd..f672ed2b1251fc 100644
--- a/llvm/unittests/Transforms/Instrumentation/MemProfUseTest.cpp
+++ b/llvm/unittests/Transforms/Instrumentation/MemProfUseTest.cpp
@@ -26,7 +26,6 @@ using namespace llvm;
 using namespace llvm::memprof;
 using testing::Contains;
 using testing::ElementsAre;
-using testing::FieldsAre;
 using testing::Pair;
 using testing::SizeIs;
 using testing::UnorderedElementsAre;

@kazutakahirata kazutakahirata merged commit 81c2024 into llvm:main Nov 20, 2024
7 of 9 checks passed
@kazutakahirata kazutakahirata deleted the memprof_cleanup_FieldsAre branch November 20, 2024 17:10
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Projects

None yet

Development

Successfully merging this pull request may close these issues.

3 participants