[PATCH] D43484: Inline applyRelocations into writeTo.

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Feb 19 20:39:50 PST 2018


ruiu closed this revision.
ruiu added a comment.

The patch to remove OutRelocation also contained this change, so this change was submitted as part of that. I do prefer shorter functions, but in this case it feels to me that writeTo does too less. That's probably a personal preference though.


https://reviews.llvm.org/D43484





More information about the llvm-commits mailing list