[PATCH] D153833: [clang][dataflow] Use namespace qualifiers when defining functions.
Martin Böhme via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Mon Jun 26 21:42:08 PDT 2023
mboehme created this revision.
Herald added subscribers: martong, xazax.hun.
Herald added a reviewer: NoQ.
Herald added a project: All.
mboehme requested review of this revision.
Herald added a project: clang.
Herald added a subscriber: cfe-commits.
See
https://llvm.org/docs/CodingStandards.html#use-namespace-qualifiers-to-implement-previously-declared-functions
Thank you to MaskRay for pointing this out on
https://reviews.llvm.org/D153006
Repository:
rG LLVM Github Monorepo
https://reviews.llvm.org/D153833
Files:
clang/lib/Analysis/FlowSensitive/RecordOps.cpp
Index: clang/lib/Analysis/FlowSensitive/RecordOps.cpp
===================================================================
--- clang/lib/Analysis/FlowSensitive/RecordOps.cpp
+++ clang/lib/Analysis/FlowSensitive/RecordOps.cpp
@@ -14,11 +14,9 @@
#define DEBUG_TYPE "dataflow"
-namespace clang {
-namespace dataflow {
-
-void copyRecord(AggregateStorageLocation &Src, AggregateStorageLocation &Dst,
- Environment &Env) {
+void clang::dataflow::copyRecord(AggregateStorageLocation &Src,
+ AggregateStorageLocation &Dst,
+ Environment &Env) {
LLVM_DEBUG({
if (Dst.getType().getCanonicalType().getUnqualifiedType() !=
Src.getType().getCanonicalType().getUnqualifiedType()) {
@@ -62,9 +60,10 @@
}
}
-bool recordsEqual(const AggregateStorageLocation &Loc1, const Environment &Env1,
- const AggregateStorageLocation &Loc2,
- const Environment &Env2) {
+bool clang::dataflow::recordsEqual(const AggregateStorageLocation &Loc1,
+ const Environment &Env1,
+ const AggregateStorageLocation &Loc2,
+ const Environment &Env2) {
LLVM_DEBUG({
if (Loc2.getType().getCanonicalType().getUnqualifiedType() !=
Loc1.getType().getCanonicalType().getUnqualifiedType()) {
@@ -124,6 +123,3 @@
return true;
}
-
-} // namespace dataflow
-} // namespace clang
-------------- next part --------------
A non-text attachment was scrubbed...
Name: D153833.534832.patch
Type: text/x-patch
Size: 1495 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20230627/11527fae/attachment-0001.bin>
More information about the cfe-commits
mailing list