[PATCH] D15045: [ELF/AArch64] Rearrange AArch64 relocations in alphabetical order. NFC.

Ed Maste via llvm-commits llvm-commits at lists.llvm.org
Fri Nov 27 10:46:33 PST 2015


emaste added a subscriber: emaste.
emaste added a comment.

In http://reviews.llvm.org/D15045#297899, @ruiu wrote:

> Should we instead order them alphabetically?


I'd certainly prefer alpha order rather than an arbitrary order that's the result of adding new ones at the end. But I wonder if there's a reasonable grouping that's lost in enforcing alpha order? Even with some relocations in separate documents we could produce a reasonable grouping - follow the order in the main spec, followed by the TLS spec for example.


http://reviews.llvm.org/D15045





More information about the llvm-commits mailing list