[llvm-commits] fix egrep options for Darwin in utils/llvmgrep

Chris Lattner clattner at apple.com
Thu Jan 26 12:31:19 PST 2012


On Jan 24, 2012, at 6:01 PM, Liang Wang wrote:
> Hi,
> 
> Currently, llvmgrep shows only filename since it uses -l as egrep
> option on Darwin.  It should use -H as for Linux.
> The following patch fixes the problem.  OK for trunk?

Sure, please commit.

-Chris

> 
> Thanks,
> Liang.
> 
> 
> Index: utils/llvmgrep
> ===================================================================
> --- utils/llvmgrep	(revision 148897)
> +++ utils/llvmgrep	(working copy)
> @@ -29,7 +29,7 @@
>   cd $TOPDIR
>   case `uname -s` in
>     SunOS) grep_cmd="ggrep -H -n" ;;
> -    Linux) grep_cmd="egrep -H -n" ;;
> +    Linux|Darwin) grep_cmd="egrep -H -n" ;;
>     *) grep_cmd="egrep -l -n" ;;
>   esac
>   ./utils/llvmdo -topdir "$TOPDIR" \
> <egrep-options-for-darwin.patch>_______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits




More information about the llvm-commits mailing list