[PATCH] D26621: [AArch64] Refactor LSE support as feature separate from V8.1a support.
Renato Golin via Phabricator via llvm-commits
llvm-commits at lists.llvm.org
Fri Nov 25 08:59:21 PST 2016
rengolin added a comment.
In https://reviews.llvm.org/D26621#605013, @t.p.northover wrote:
> OK, I eventually managed to track it down to "large system extensions", I think we should include that somewhere. Probably alongside noting that it's the atomics because the link is decidedly non-obvious even with the expanded name.
I think the current description is better than if it just said "Large System Extension", as it actually tells you what it is about. :)
I don't mind as it is, or an additional comment just about the declaration. Everything else looks fine, too. Leaving to Tim to approve.
cheers,
--renato
https://reviews.llvm.org/D26621
More information about the llvm-commits
mailing list