[PATCH] cmake: Find xdot or xdot.py

Chandler Carruth chandlerc at gmail.com
Mon Jul 8 13:05:54 PDT 2013


On Mon, Jul 8, 2013 at 1:01 PM, Matt Arsenault <Matthew.Arsenault at amd.com>wrote:

> It uses the first one as the basis name of the variables, so trying xdot
> first will name it HAVE_XDOT/LLVM_PATH_XDOT instead of
> HAVE_XDOT_PY/LLVM_PATH_XDOT_PY, and it's classier to not include the file
> extension in the program name.


My question was -- why move this one line down, not why list the arguments
in this order. I agree with your reason for the latter. =] Anyways, its a
minor point, commit whenever.
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130708/75ae5c49/attachment.html>


More information about the llvm-commits mailing list