Page Menu
Home
Phabricator
This is an archive of the discontinued LLVM Phabricator instance.
Paths
Table of Contents
t
-
cfe/trunk/
-
trunk/
-
include/clang/AST/
-
clang/
-
AST/
-
NestedNameSpecifier.h
-
lib/AST/
-
AST/
-
NestedNameSpecifier.cpp
Hide Panel
f
Keyboard Reference
?
Differential
D52870
[NestedNameSpecifier] Add missing stream-specific dump methods
Closed
Public
Authored by
steveire
on Oct 4 2018, 1:51 AM.
Download Raw Diff
Details
Reviewers
aaron.ballman
Commits
rG07dd5af6dfae: [NestedNameSpecifier] Add missing stream-specific dump methods
rC343807: [NestedNameSpecifier] Add missing stream-specific dump methods
rL343807: [NestedNameSpecifier] Add missing stream-specific dump methods
Diff Detail
Repository
rL LLVM
Event Timeline
steveire
created this revision.
Oct 4 2018, 1:51 AM
Herald
added a subscriber:
cfe-commits
.
·
View Herald Transcript
Oct 4 2018, 1:51 AM
Harbormaster
completed remote builds in
B23442: Diff 168241
.
Oct 4 2018, 1:51 AM
aaron.ballman
added inline comments.
Oct 4 2018, 11:12 AM
lib/AST/NestedNameSpecifier.cpp
342 ↗
(On Diff #168241)
LLVM_DUMP_METHOD
?
346 ↗
(On Diff #168241)
LLVM_DUMP_METHOD
?
350 ↗
(On Diff #168241)
LLVM_DUMP_METHOD
?
351 ↗
(On Diff #168241)
Formatting is incorrect here.
356–357 ↗
(On Diff #168241)
Same here.
steveire
updated this revision to
Diff 168338
.
Oct 4 2018, 11:30 AM
Comment Actions
Format
Harbormaster
completed remote builds in
B23468: Diff 168338
.
Oct 4 2018, 11:30 AM
steveire
marked 5 inline comments as done.
Oct 4 2018, 11:31 AM
aaron.ballman
accepted this revision.
Oct 4 2018, 12:20 PM
Comment Actions
LGTM!
This revision is now accepted and ready to land.
Oct 4 2018, 12:20 PM
Closed by commit
rL343807: [NestedNameSpecifier] Add missing stream-specific dump methods
(authored by
steveire
).
·
Explain Why
Oct 4 2018, 12:24 PM
This revision was automatically updated to reflect the committed changes.
Herald
added a subscriber:
llvm-commits
.
·
View Herald Transcript
Oct 4 2018, 12:24 PM