[LLVMdev] [PATCH]: Add SparseBitmap implementation

Devang Patel dpatel at apple.com
Tue Sep 4 19:20:43 PDT 2007


On Sep 4, 2007, at 4:36 PM, Daniel Berlin wrote:

> Also, someone on IRC asked about the weird function naming
> conventions.  I know most of llvm uses mixedCase, but i copied the
> style of BitVector, which has a weird mix.  I'm happy to change it :)

Yes please. By default, it is a good idea to follow llvm coding  
conventions instead of copying violation of coding style from  
existing code. Otherwise it pollutes code and, in future, gives  
someone one additional reason to not follow preferred coding  
style  :) While you're here please fix BitVector also.

Thanks!
-
Devang
-------------- next part --------------
An HTML attachment was scrubbed...
URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20070904/efe1520f/attachment.html>


More information about the llvm-dev mailing list