[PATCH] [Object] Split the ELF interface into 3 parts.
Sean Silva
silvas at purdue.edu
Tue Jun 25 14:13:21 PDT 2013
Is there any way you can factor out ELFTypes.h in a separate (obvious) patch? In general, I'm having a hard time seeing what is "new code" and what is code that is just being moved around and renamed; can you split the patch up at all?
================
Comment at: include/llvm/Object/ELF.h:357
@@ -780,1 +356,3 @@
+ sec->sh_entsize,
+ (const char *)(base() + sec->sh_offset + sec->sh_size));
}
----------------
It seems like you have a lot of cases of this pattern `(const char *)(base() + X))`. Could you factor that out into a member function?
http://llvm-reviews.chandlerc.com/D1033
More information about the llvm-commits
mailing list