[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