[PATCH] TableGen backend to generate an index in ctags(1) format.

Sean Silva silvas at purdue.edu
Tue Mar 19 19:31:22 PDT 2013


On Tue, Mar 19, 2013 at 5:05 PM, Jakob Stoklund Olesen <stoklund at 2pi.dk>wrote:

>
> On Mar 19, 2013, at 11:08 AM, Stefanus Du Toit <stefanus.du.toit at intel.com>
> wrote:
>
> >
> >  If the script works today, rewriting it in Python now seems
> unnecessary. Sounds like Kevin will use Python for future scripts, but
> since this is just a little utility, I think it's fine.
>
> +1
>
> That shell script looks much nicer than most of our Python scripts.
>
> /jakob
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
>

Alright it looks like there is consensus to put the script in as-is.

Kevin, please update the patch to address my comment about x-ref'ing the
script from the file header, and the typo I just caught (on phab). Do you
need me to commit this for you?

Now that I think about it, AFAIK currently the only LLVM development
environment where a POSIX-compatible shell would not be available is for
people developing in Visual Studio, and I don't think they are very
interested in ctags :P

Also, Kevin, would you care to enlighten us mere mortals where you learned
the black art of writing readable shell scripts?

-- Sean Silva
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20130319/8fcaf7ea/attachment.html>


More information about the llvm-commits mailing list