[Lldb-commits] [PATCH] D65862: Remove xcode-specific Config.h

2019-08-08 Thread Pavel Labath via Phabricator via lldb-commits
This revision was automatically updated to reflect the committed changes. Closed by commit rL368266: Remove xcode-specific Config.h (authored by labath, committed by ). Herald added a project: LLVM. Herald added a subscriber: llvm-commits. Repository: rL LLVM CHANGES SINCE LAST ACTION https:

[Lldb-commits] [PATCH] D65862: Remove xcode-specific Config.h

2019-08-07 Thread Stefan Gränitz via Phabricator via lldb-commits
sgraenitz accepted this revision. sgraenitz added a comment. This revision is now accepted and ready to land. Right CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65862/new/ https://reviews.llvm.org/D65862 ___ lldb-commits mailing list lldb-

[Lldb-commits] [PATCH] D65862: Remove xcode-specific Config.h

2019-08-07 Thread Pavel Labath via Phabricator via lldb-commits
labath added a comment. In D65862#1618740 , @sgraenitz wrote: > Originally introduced with D31969 . LGTM. > Maybe we can remove even more? (here or in future commits) I don't see what more could be removed here. Most of

[Lldb-commits] [PATCH] D65862: Remove xcode-specific Config.h

2019-08-07 Thread Stefan Gränitz via Phabricator via lldb-commits
sgraenitz added a comment. Originally introduced with D31969 . LGTM. Maybe we can remove even more? CHANGES SINCE LAST ACTION https://reviews.llvm.org/D65862/new/ https://reviews.llvm.org/D65862 ___ lldb-commits

[Lldb-commits] [PATCH] D65862: Remove xcode-specific Config.h

2019-08-07 Thread Pavel Labath via Phabricator via lldb-commits
labath created this revision. labath added reviewers: sgraenitz, beanz. Herald added a subscriber: mgorny. Now that the xcode project is removed, we no longer need/use the hand-maintained Config.h file, as everything is configured through cmake. This patch deletes that file and reverts some of th