[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