[PATCH] D38407: [ELF] - Do --hash-style=both by default.

Rui Ueyama via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 29 12:37:20 PDT 2017


ruiu added a comment.

Could you start a thread in the mailing list to propose a change of the default settings first then create a patch?

I don't want you to update that many tests (which probably needed a lot of labor work) before getting any consensus, even if reaching a consensus is obvious to you. A rule of thumb is that, code review is not a place to discuss policy changes.


https://reviews.llvm.org/D38407





More information about the llvm-commits mailing list