[PATCH] D28092: [ELF] - Specify -section argument for objdump calls in testcases.

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 3 16:02:41 PST 2017


ruiu added a comment.

I think I agree with Davide. This patch might not necessarily be a bad change, but to me it seems to be adding extraneous command line options to so many command invocations. As the tests work without these command line options, I wouldn't add that based on Occum's razor-ish principle. I want to keep it simple.


https://reviews.llvm.org/D28092





More information about the llvm-commits mailing list