Is it kosher to include vim mode line comments inside of LLVM source files? I would like to do this inside of the MicroBlaze backend to ensure that tabs are expanded into exactly two spaces. I see that right now the following files have these vim mode line comments: include/llvm/ADT/SetVector.h lib/Archive/ArchiveInternals.h lib/Linker/LinkModules.cpp lib/Transforms/IPO/DeadTypeElimination.cpp lib/VMCore/Function.cpp Since this does not seem to be a wide spread practice I thought I would ask before I made the changes. -- Wesley Peck University of Kansas SLDG Laboratory
On Feb 26, 2010, at 11:48 AM, Wesley Peck wrote:> Is it kosher to include vim mode line comments inside of LLVM source files? I would like to do this inside of the MicroBlaze backend to ensure that tabs are expanded into exactly two spaces. I see that right now the following files have these vim mode line comments: > include/llvm/ADT/SetVector.h > lib/Archive/ArchiveInternals.h > lib/Linker/LinkModules.cpp > lib/Transforms/IPO/DeadTypeElimination.cpp > lib/VMCore/Function.cpp > > Since this does not seem to be a wide spread practice I thought I would ask before I made the changes.I'd prefer not to do that, but if it is really useful to you, go for it in the mblaze files only. -Chris> > -- > Wesley Peck > University of Kansas > SLDG Laboratory > > > _______________________________________________ > LLVM Developers mailing list > LLVMdev at cs.uiuc.edu http://llvm.cs.uiuc.edu > http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev
utils/vim/vimrc is the usual way to use vim with LLVM source. Is that inconvenient? Dan On Feb 26, 2010, at 11:48 AM, Wesley Peck <peckw at ittc.ku.edu> wrote:> Is it kosher to include vim mode line comments inside of LLVM source > files? I would like to do this inside of the MicroBlaze backend to > ensure that tabs are expanded into exactly two spaces. I see that > right now the following files have these vim mode line comments: > include/llvm/ADT/SetVector.h > lib/Archive/ArchiveInternals.h > lib/Linker/LinkModules.cpp > lib/Transforms/IPO/DeadTypeElimination.cpp > lib/VMCore/Function.cpp > > Since this does not seem to be a wide spread practice I thought I > would ask before I made the changes. > > -- > Wesley Peck > University of Kansas > SLDG Laboratory > > > _______________________________________________ > LLVM Developers mailing list > LLVMdev at cs.uiuc.edu http://llvm.cs.uiuc.edu > http://lists.cs.uiuc.edu/mailman/listinfo/llvmdev
Seemingly Similar Threads
- [LLVMdev] Unreachable executed with fast Regalloc and Sparc backend
- [LLVMdev] [PATCH] Configurable machine type in ELFObjectWriter
- [LLVMdev] Unreachable executed with fast Regalloc and Sparc backend
- [LLVMdev] MC ELFObjectWriter backend refactoring
- [LLVMdev] Branch delay slots broken.