[PATCH] D13397: Rename alias definitions by both aliased and alias name

Ed Maste via llvm-commits llvm-commits at lists.llvm.org
Fri Oct 2 13:58:07 PDT 2015


emaste added a comment.

In http://reviews.llvm.org/D13397#258935, @rafael wrote:

> Why do we need both? Why not, for example
>
> def alias_library_path ....


That was my original thought, but then I noticed that the existing Bdynamic... ones have both. That approach has the nice property that multiple aliases for the same option are grouped together when sorting them alphabetically.


http://reviews.llvm.org/D13397





More information about the llvm-commits mailing list