[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