We can leverage JSType again by adding a new value for NativeExecutable, which is the only class that overrides intrinsic.
Created attachment 115450 [details] Patch
Comment on attachment 115450 [details] Patch View in context: https://bugs.webkit.org/attachment.cgi?id=115450&action=review r=me > Source/JavaScriptCore/runtime/Executable.cpp:57 > + if (structure()->typeInfo().type() == NativeExecutableType) > + return jsCast<const NativeExecutable*>(this)->intrinsic(); Could just use classInfo here instead of a type flag. I mention this because it might be nice to deploy jsDynamicCast here, and I assume jsDynamicCast would be based on ClassInfo. > Source/JavaScriptCore/runtime/JSType.h:54 > }; Do we really need to spell these constants out explicitly? Do their bits have any meaning?
Created attachment 115486 [details] Patch
Comment on attachment 115486 [details] Patch Clearing flags on attachment: 115486 Committed r100810: <http://trac.webkit.org/changeset/100810>
All reviewed patches have been landed. Closing bug.