[PATCH] [lld] add calls to nativereader/nativewriter

Rui Ueyama ruiu at google.com
Thu Oct 17 16:29:08 PDT 2013


  I think you forgot to add RoundTripNativePass.cpp and RoundTripYAMLPass.cpp to this patch.


================
Comment at: include/lld/Core/Resolver.h:85
@@ -83,3 +84,3 @@
 
-  class MergedFile : public MutableFile {
+  class MergedFile : public SimpleFile {
   public:
----------------
This change makes SimpleFile class the sole child class of MutableFile. I'd merge them into one class to reduce the number of classes and complexity.

================
Comment at: include/lld/Passes/RoundTripNativePass.h:10
@@ +9,3 @@
+
+#ifndef LLD_PASSES_NATIVE_PASS_H
+#define LLD_PASSES_NATIVE_PASS_H
----------------
This should be LLD_PASSES_ROUND_TRIP_NATIVE_PASS_H.

================
Comment at: include/lld/Passes/RoundTripYAMLPass.h:10
@@ +9,3 @@
+
+#ifndef LLD_PASSES_ROUNDTRIP_YAML_PASS_H
+#define LLD_PASSES_ROUNDTRIP_YAML_PASS_H
----------------
This should be LLD_PASSES_ROUND_TRIP_YAML_PASS_H.

================
Comment at: lib/Driver/Driver.cpp:130
@@ -117,1 +129,3 @@
 
+  lld::File *resultFile = &merged;
+
----------------
Why do you need this temporary variable?


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



More information about the llvm-commits mailing list