Bug 312240 (MONO41990) - [PATCH] Support for flagging `pinned' variable.s
Summary: [PATCH] Support for flagging `pinned' variable.s
Status: RESOLVED FIXED
Alias: MONO41990
Product: Mono: Compilers
Classification: Mono
Component: C# (show other bugs)
Version: unspecified
Hardware: Other Other
: P3 - Medium : Normal
Target Milestone: ---
Assignee: Ben Maurer
QA Contact: Mono Bugs
URL:
Whiteboard:
Keywords: patch
Depends on:
Blocks:
 
Reported: 2003-04-27 20:26 UTC by Miguel de Icaza
Modified: 2007-09-15 21:24 UTC (History)
0 users

See Also:
Found By: ---
Services Priority:
Business Priority:
Blocker: ---
Marketing QA Status: ---
IT Deployment: ---


Attachments
patch for mcs (971 bytes, patch)
2004-03-24 07:30 UTC, Thomas Wiest
Details | Diff
patch for mono/metadata (2.24 KB, patch)
2004-03-24 07:30 UTC, Thomas Wiest
Details | Diff
patch for mcs/class/corlib/System.Reflection.Emit (541 bytes, patch)
2004-03-24 07:30 UTC, Thomas Wiest
Details | Diff

Note You need to log in before you can comment on or make changes to this bug.
Description Thomas Wiest 2007-09-15 18:09:12 UTC


---- Reported by miguel@ximian.com 2003-04-27 13:26:48 MST ----

This is another topic for discussion.

We need a Mono-specific call to flag a LocalBuilder variable as "pinned",
so that the generated code will properly flag this slot.

This is not important for Mono, given that we use the Boehm GC, but it will
be important to move the code we generate over to the MS.NET

This time I believe its ok to use a dynamic invocation, as this is rarely
used, its only used when doing a "fixed" statement.



---- Additional Comments From bmaurer@users.sf.net 2004-03-24 00:30:01 MST ----

Created an attachment (id=164636)
patch for mcs




---- Additional Comments From bmaurer@users.sf.net 2004-03-24 00:30:28 MST ----

Created an attachment (id=164637)
patch for mono/metadata




---- Additional Comments From bmaurer@users.sf.net 2004-03-24 00:30:58 MST ----

Created an attachment (id=164638)
patch for mcs/class/corlib/System.Reflection.Emit




---- Additional Comments From bmaurer@users.sf.net 2004-03-24 00:31:34 MST ----

The attached patches should add support for pinned variables.



---- Additional Comments From miguel@ximian.com 2004-03-29 22:21:15 MST ----

Dude, could you redo those patches without the property, but just
a plain method, that would get the job done?

SetPinned() is all we need, we do not need to find out if something
is pinned.



---- Additional Comments From bmaurer@users.sf.net 2004-03-29 23:13:02 MST ----

Fixed in CVS!

(WOOOHOOOOO!)

Imported an attachment (id=164636)
Imported an attachment (id=164637)
Imported an attachment (id=164638)

Unknown operating system unknown. Setting to default OS "Other".