[PATCH] Add a dereferencable attribute

Jan Voung jvoung at chromium.org
Fri Jul 11 11:21:26 PDT 2014


================
Comment at: include/llvm/Bitcode/LLVMBitCodes.h:378
@@ -378,1 +377,3 @@
+    ATTR_KIND_JUMP_TABLE = 40,
+    ATTR_KIND_DEFERERENCEABLE = 41
   };
----------------
DEFERERENCEABLE -> DEREFERENCEABLE ?

================
Comment at: include/llvm/IR/Attributes.h:91
@@ -90,2 +90,3 @@
     NonNull,               ///< Pointer is known to be not null
+    Dereferenceable,       ///< Pointer is known to be dereferenceable
     NoRedZone,             ///< Disable redzone
----------------
JF Bastien wrote:
> You'll make some people sad (not PNaCl, but Renderscript and others) by adding the attribute here in the enum instead of at the end, since it changes bitcode slightly.
I think the enum in the bitc:: namespace is the one that affects serialized bitcode and not this enum?

http://reviews.llvm.org/D4449






More information about the llvm-commits mailing list