This revision was automatically updated to reflect the committed changes.
Closed by commit rGd1a1798e51a3: [lldb] Replace unneeded use of Foundation with
ObjectiveC in tests (NFC) (authored by kastiglione).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.or
Author: Dave Lee
Date: 2021-04-04T20:04:41-07:00
New Revision: d1a1798e51a30fbf537e9fd8931a49b504f37a25
URL:
https://github.com/llvm/llvm-project/commit/d1a1798e51a30fbf537e9fd8931a49b504f37a25
DIFF:
https://github.com/llvm/llvm-project/commit/d1a1798e51a30fbf537e9fd8931a49b504f37a25.diff
LOG:
This revision was automatically updated to reflect the committed changes.
Closed by commit rG10ed479d7303: [lldb] Import ObjectiveC module instead of
Foundation in test (authored by kastiglione).
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D99859/ne
Author: Dave Lee
Date: 2021-04-04T20:04:26-07:00
New Revision: 10ed479d7303af7c6e2528185c7e7cd318c9172c
URL:
https://github.com/llvm/llvm-project/commit/10ed479d7303af7c6e2528185c7e7cd318c9172c
DIFF:
https://github.com/llvm/llvm-project/commit/10ed479d7303af7c6e2528185c7e7cd318c9172c.diff
LOG:
kastiglione added inline comments.
Comment at: lldb/test/API/commands/frame/recognizer/main.m:1
-#import
+#import
teemperor wrote:
> teemperor wrote:
> > I guess this could also be just `stdio.h` (FWIW, the printfs in tests can
> > also be removed as they ju
kastiglione updated this revision to Diff 335187.
kastiglione added a comment.
fix lldb/test/API/commands/frame/recognizer/main.m
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D99867/new/
https://reviews.llvm.org/D99867
Files:
lldb/test/API/comma
teemperor added inline comments.
Comment at: lldb/test/API/commands/frame/recognizer/main.m:1
-#import
+#import
teemperor wrote:
> I guess this could also be just `stdio.h` (FWIW, the printfs in tests can
> also be removed as they just require including syst
teemperor accepted this revision.
teemperor added a comment.
This revision is now accepted and ready to land.
Herald added a subscriber: JDevlieghere.
Thanks for cleaning this up!
Comment at: lldb/test/API/commands/frame/recognizer/main.m:1
-#import
+#import
---
kastiglione created this revision.
kastiglione added a reviewer: teemperor.
kastiglione requested review of this revision.
Herald added a project: LLDB.
Herald added a subscriber: lldb-commits.
When referencing `NSObject`, it's enough to import `objc/NSObject.h`. Importing
`Foundation` is unneces
mgorny created this revision.
mgorny added reviewers: labath, emaste, krytarowski.
mgorny requested review of this revision.
Initial work on moving fork support from server to gdb-remote protocol. If
`fork-events` (`vfork-events`) extension is supported, the server reports them
to the client as
kastiglione created this revision.
kastiglione added a reviewer: teemperor.
kastiglione requested review of this revision.
Herald added a project: LLDB.
Herald added a subscriber: lldb-commits.
Use `@import ObjectiveC` instead of `@import Foundation`, as the former is all
that's needed, and result
Author: Jason Molenda
Date: 2021-04-04T01:47:35-07:00
New Revision: 602ab188a7e18b97d9af95e17271e8fbee129081
URL:
https://github.com/llvm/llvm-project/commit/602ab188a7e18b97d9af95e17271e8fbee129081
DIFF:
https://github.com/llvm/llvm-project/commit/602ab188a7e18b97d9af95e17271e8fbee129081.diff
12 matches
Mail list logo