[PATCH] D16468: [ELF] - Attempt to simplify the relocations relaxation code

George Rimar via llvm-commits llvm-commits at lists.llvm.org
Fri Jan 22 14:08:02 PST 2016


grimar abandoned this revision.
grimar added a comment.

I will wait to see better solution. I am sure it exist.


================
Comment at: ELF/InputSection.h:85-87
@@ -81,1 +84,5 @@
 
+  struct RelocData {
+    bool Optimized = false;
+  };
+  struct RelocSec {
----------------
ruiu wrote:
> Why do you need a struct that has only one member?
Please read patch notes, I am not sure will we need any additional data or not. So it is easy to replace it with single bool.


http://reviews.llvm.org/D16468





More information about the llvm-commits mailing list