[PATCH] [lld][Driver] Replace Target with TargetInfo. Simplify LinkerInput.

[Shankar Kalpathi Easwaran]

================
Comment at: tools/lld-core/lld-core.cpp:199-205
@@ +198,9 @@
+
+  virtual ErrorOr<Reader &> getReader(const LinkerInput &input) const {
+    llvm_unreachable("Unimplemented!");
+  }
+
+  virtual ErrorOr<Writer &> getWriter() const {
+    llvm_unreachable("Unimplemented!");
+  }
+
----------------
Isnt this supposed to use ReaderYAML/WriterYAML ?

================
Comment at: tools/lld-core/lld-core.cpp:301-306
@@ -291,8 +300,8 @@
       break;
     case readerELF:
       reader = createReaderELF(*eti,
       [&] (const LinkerInput &) -> ErrorOr<Reader&> {
         return *reader;
       });
       break;
     default:
----------------
Why are these references still there ?

================
Comment at: lib/Driver/LinkerInvocation.cpp:24-27
@@ -23,1 +23,6 @@
 
+namespace {
+std::unique_ptr<TargetInfo> createTargetInfo(const LinkerOptions &lo) {
+  return ELFTargetInfo::create(lo);
+}
+}
----------------
Why is it hardcoded to create ELFTargetInfo ?

================
Comment at: include/lld/ReaderWriter/MachOTargetInfo.h:33-39
@@ -30,1 +32,9 @@
 
+  virtual ErrorOr<Reader &> getReader(const LinkerInput &input) const {
+    llvm_unreachable("Unimplemented!");
+  }
+
+  virtual ErrorOr<Writer &> getWriter() const {
+    llvm_unreachable("Unimplemented!");
+  }
+
----------------
These should be changed to an error code instead of llvm_unreachable.


http://llvm-reviews.chandlerc.com/D372