[cfe-commits] r84328 - /cfe/trunk/include/clang/AST/Attr.h

Daniel Dunbar daniel at zuster.org
Sat Oct 17 02:39:11 PDT 2009


Author: ddunbar
Date: Sat Oct 17 04:38:57 2009
New Revision: 84328

URL: http://llvm.org/viewvc/llvm-project?rev=84328&view=rev
Log:
Remove a FIXME that doesn't belong, there are numerous other places that use
alignment in bits in clang. If we really want to fix this, it should be a bugzilla.

Modified:
    cfe/trunk/include/clang/AST/Attr.h

Modified: cfe/trunk/include/clang/AST/Attr.h
URL: http://llvm.org/viewvc/llvm-project/cfe/trunk/include/clang/AST/Attr.h?rev=84328&r1=84327&r2=84328&view=diff

==============================================================================
--- cfe/trunk/include/clang/AST/Attr.h (original)
+++ cfe/trunk/include/clang/AST/Attr.h Sat Oct 17 04:38:57 2009
@@ -185,7 +185,6 @@
 public:
   AlignedAttr(unsigned alignment) : Attr(Aligned), Alignment(alignment) {}
 
-  // FIXME: Should use addressable units, not bits, to match llvm
   /// getAlignment - The specified alignment in bits.
   unsigned getAlignment() const { return Alignment; }
 





More information about the cfe-commits mailing list