[PATCH] [PlaceSafepoints] New attributes for patchable statepoints.

Pat Gavlin pagavlin at microsoft.com
Wed May 13 10:46:34 PDT 2015


It would be nice to have documentation for these attributes somewhere.


================
Comment at: lib/Transforms/Scalar/PlaceSafepoints.cpp:908-911
@@ +907,6 @@
+  AttributeSet AS = CS.getAttributes();
+  Attribute AttrID =
+      AS.getAttribute(AttributeSet::FunctionIndex, "statepoint-id");
+  Attribute AttrNumPatchBytes = AS.getAttribute(AttributeSet::FunctionIndex,
+                                                "statepoint-num-patch-bytes");
+
----------------
Are these attributes propagated by PlaceSafepoints? If so, should they be dropped instead? I don't necessarily see any reason to drop them; just wondering if this was considered.

http://reviews.llvm.org/D9735

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list