//
// Author:
// Martin Baulig (martin@ximian.com)
+// Raja R Harinath (rharinath@novell.com)
//
// (C) 2001, 2002, 2003 Ximian, Inc.
//
Switch,
// Switch section.
- SwitchSection
+ SwitchSection,
+
+ // The toplevel block of a function
+ Toplevel
}
// <summary>
switch (type) {
case BranchingType.Exception:
case BranchingType.Labeled:
+ case BranchingType.Toplevel:
throw new InvalidOperationException ();
case BranchingType.Switch:
// </summary>
public readonly Location Location;
- //
- // Private
- //
- VariableMap param_map, local_map;
+ protected VariableMap param_map, local_map;
static int next_id = 0;
int id;
// </summary>
public readonly Block Block;
- // <summary>
- // If this is true, then the usage vector has been modified and must be
- // merged when we're done with this branching.
- // </summary>
- public bool IsDirty;
-
// <summary>
// The number of parameters in this block.
// </summary>
//
// Normally, you should not use any of these constructors.
//
- public UsageVector (SiblingType type, UsageVector parent,
- Block block, Location loc,
- int num_params, int num_locals)
+ public UsageVector (SiblingType type, UsageVector parent, Block block, Location loc, int num_params, int num_locals)
{
this.Type = type;
this.Block = block;
this.CountParameters = num_params;
this.CountLocals = num_locals;
- if (parent != null) {
- if (num_locals > 0)
- locals = new MyBitVector (parent.locals, CountLocals);
-
- if (num_params > 0)
- parameters = new MyBitVector (parent.parameters, num_params);
+ locals = num_locals == 0
+ ? MyBitVector.Empty
+ : new MyBitVector (parent == null ? MyBitVector.Empty : parent.locals, num_locals);
- reachability = parent.Reachability.Clone ();
- } else {
- if (num_locals > 0)
- locals = new MyBitVector (null, CountLocals);
-
- if (num_params > 0)
- parameters = new MyBitVector (null, num_params);
+ parameters = num_params == 0
+ ? MyBitVector.Empty
+ : new MyBitVector (parent == null ? MyBitVector.Empty : parent.parameters, num_params);
- reachability = Reachability.Always ();
- }
+ reachability = parent == null ? Reachability.Always () : parent.Reachability.Clone ();
id = ++next_id;
}
- public UsageVector (SiblingType type, UsageVector parent,
- Block block, Location loc)
- : this (type, parent, block, loc,
- parent.CountParameters, parent.CountLocals)
+ public UsageVector (SiblingType type, UsageVector parent, Block block, Location loc)
+ : this (type, parent, block, loc, parent.CountParameters, parent.CountLocals)
{ }
- public UsageVector (MyBitVector parameters, MyBitVector locals,
- Reachability reachability, Block block,
- Location loc)
+ public UsageVector (MyBitVector parameters, MyBitVector locals, Reachability reachability, Block block, Location loc)
{
this.Type = SiblingType.Block;
this.Location = loc;
// </summary>
public UsageVector Clone ()
{
- UsageVector retval = new UsageVector (
- Type, null, Block, Location,
- CountParameters, CountLocals);
+ UsageVector retval = new UsageVector (Type, null, Block, Location, CountParameters, CountLocals);
- if (retval.locals != null)
- retval.locals = locals.Clone ();
-
- if (parameters != null)
- retval.parameters = parameters.Clone ();
-
+ retval.locals = locals.Clone ();
+ retval.parameters = parameters.Clone ();
retval.reachability = reachability.Clone ();
return retval;
if (!var.IsParameter && Reachability.IsUnreachable)
return;
- IsDirty = true;
var.SetAssigned (var.IsParameter ? parameters : locals);
}
if (!var.IsParameter && Reachability.IsUnreachable)
return;
- IsDirty = true;
var.SetFieldAssigned (var.IsParameter ? parameters : locals, name);
}
public void Return ()
{
- if (!reachability.IsUnreachable) {
- IsDirty = true;
+ if (!reachability.IsUnreachable)
reachability.SetReturns ();
- }
}
public void Throw ()
{
if (!reachability.IsUnreachable) {
- IsDirty = true;
reachability.SetThrows ();
reachability.SetBarrier ();
}
public void Goto ()
{
- if (!reachability.IsUnreachable) {
- IsDirty = true;
+ if (!reachability.IsUnreachable)
reachability.SetBarrier ();
+ }
+
+ public static UsageVector MergeSiblings (UsageVector sibling_list, Location loc)
+ {
+ if (sibling_list.Next == null)
+ return sibling_list;
+
+ MyBitVector locals = null;
+ MyBitVector parameters = null;
+ Reachability reachability = null;
+
+ for (UsageVector child = sibling_list; child != null; child = child.Next) {
+ Report.Debug (2, " MERGING SIBLING ", reachability, child);
+
+ if (reachability == null)
+ reachability = child.Reachability.Clone ();
+ else
+ reachability.Meet (child.Reachability);
+
+ // A local variable is initialized after a flow branching if it
+ // has been initialized in all its branches which do neither
+ // always return or always throw an exception.
+ //
+ // If a branch may return, but does not always return, then we
+ // can treat it like a never-returning branch here: control will
+ // only reach the code position after the branching if we did not
+ // return here.
+ //
+ // It's important to distinguish between always and sometimes
+ // returning branches here:
+ //
+ // 1 int a;
+ // 2 if (something) {
+ // 3 return;
+ // 4 a = 5;
+ // 5 }
+ // 6 Console.WriteLine (a);
+ //
+ // The if block in lines 3-4 always returns, so we must not look
+ // at the initialization of `a' in line 4 - thus it'll still be
+ // uninitialized in line 6.
+ //
+ // On the other hand, the following is allowed:
+ //
+ // 1 int a;
+ // 2 if (something)
+ // 3 a = 5;
+ // 4 else
+ // 5 return;
+ // 6 Console.WriteLine (a);
+ //
+ // Here, `a' is initialized in line 3 and we must not look at
+ // line 5 since it always returns.
+ //
+ bool unreachable = child.Reachability.IsUnreachable;
+
+ Report.Debug (2, " MERGING SIBLING #1", reachability,
+ child.Type, child.Reachability.IsUnreachable, unreachable);
+
+ if (!unreachable)
+ locals &= child.locals;
+
+ // An `out' parameter must be assigned in all branches which do
+ // not always throw an exception.
+ if (!child.Reachability.AlwaysThrows)
+ parameters &= child.parameters;
+
+ Report.Debug (2, " MERGING SIBLING #2", parameters, locals);
}
+
+ if (reachability == null)
+ throw new InternalErrorException ("Cannot happen: the loop above runs at least twice");
+
+ return new UsageVector (parameters, locals, reachability, null, loc);
}
// <summary>
// Merges a child branching.
// </summary>
- public UsageVector MergeChild (UsageVector child, bool implicit_block)
+ public UsageVector MergeChild (UsageVector child, bool overwrite)
{
- Report.Debug (2, " MERGING CHILD EFFECTS", this, child, IsDirty, reachability, Type);
+ Report.Debug (2, " MERGING CHILD EFFECTS", this, child, reachability, Type);
Reachability new_r = child.Reachability;
return child;
}
- if (locals != null && child.LocalVector != null)
- locals.Or (child.LocalVector);
-
- if (child.ParameterVector != null)
- parameters.Or (child.ParameterVector);
+ locals |= child.locals;
+ parameters |= child.parameters;
- if (implicit_block)
+ if (overwrite)
reachability = new_r.Clone ();
else
reachability.Or (new_r);
- IsDirty = true;
-
return child;
}
- // <summary>
- // Tells control flow analysis that the current code position may be reached with
- // a forward jump from any of the origins listed in `origin_vectors' which is a
- // list of UsageVectors.
- //
- // This is used when resolving forward gotos - in the following example, the
- // variable `a' is uninitialized in line 8 becase this line may be reached via
- // the goto in line 4:
- //
- // 1 int a;
- //
- // 3 if (something)
- // 4 goto World;
- //
- // 6 a = 5;
- //
- // 7 World:
- // 8 Console.WriteLine (a);
- //
- // </summary>
- public void MergeJumpOrigins (UsageVector o_vectors)
- {
- Report.Debug (1, " MERGING JUMP ORIGINS", this);
-
- if (o_vectors == null)
- return;
-
- UsageVector vector = o_vectors;
- if (reachability.IsUnreachable) {
- Report.Debug (1, " MERGING JUMP ORIGIN INTO UNREACHABLE", this, vector);
- if (locals != null && vector.Locals != null)
- locals.Or (vector.locals);
- if (parameters != null)
- parameters.Or (vector.parameters);
- reachability.Meet (vector.Reachability);
- vector = vector.Next;
- }
-
- for (; vector != null; vector = vector.Next) {
- Report.Debug (1, " MERGING JUMP ORIGIN", this, vector);
-
- if (locals != null)
- locals.And (vector.locals);
- if (parameters != null)
- parameters.And (vector.parameters);
- reachability.Meet (vector.Reachability);
-
- Report.Debug (1, " MERGING JUMP ORIGIN #1", vector);
- }
-
- Report.Debug (1, " MERGING JUMP ORIGINS DONE", this);
- }
-
public void MergeOrigins (UsageVector o_vectors)
{
Report.Debug (1, " MERGING BREAK ORIGINS", this);
if (o_vectors == null)
return;
- UsageVector vector = o_vectors;
-
if (reachability.IsUnreachable) {
- Report.Debug (1, " MERGING BREAK ORIGIN INTO UNREACHABLE", vector);
- locals = vector.Locals;
- parameters = vector.Parameters;
- reachability.Meet (vector.Reachability);
- vector = vector.Next;
+ if (locals != null)
+ locals.SetAll (true);
+ if (parameters != null)
+ parameters.SetAll (true);
}
- for (; vector != null; vector = vector.Next) {
+ for (UsageVector vector = o_vectors; vector != null; vector = vector.Next) {
Report.Debug (1, " MERGING BREAK ORIGIN", vector);
- if (locals != null && vector.locals != null)
- locals.And (vector.locals);
- if (parameters != null && vector.parameters != null)
- parameters.And (vector.parameters);
+ if (vector.Reachability.IsUnreachable)
+ continue;
+ locals &= vector.locals;
+ parameters &= vector.parameters;
reachability.Meet (vector.Reachability);
}
Report.Debug (1, " MERGING BREAK ORIGINS DONE", this);
}
- // <summary>
- // Performs an `or' operation on the locals and the parameters.
- // </summary>
- public void Or (UsageVector new_vector)
- {
- IsDirty = true;
- locals.Or (new_vector.locals);
- if (parameters != null)
- parameters.Or (new_vector.parameters);
- }
-
- // <summary>
- // Performs an `and' operation on the locals.
- // </summary>
- public void AndLocals (UsageVector new_vector)
- {
- IsDirty = true;
- locals.And (new_vector.locals);
- }
-
- public bool HasParameters {
- get { return parameters != null; }
- }
-
- public bool HasLocals {
- get { return locals != null; }
- }
-
- // <summary>
- // Returns a deep copy of the parameters.
- // </summary>
- public MyBitVector Parameters {
- get { return parameters == null ? null : parameters.Clone (); }
- }
-
- // <summary>
- // Returns a deep copy of the locals.
- // </summary>
- public MyBitVector Locals {
- get { return locals == null ? null : locals.Clone (); }
- }
-
- public MyBitVector ParameterVector {
- get { return parameters; }
- }
-
- public MyBitVector LocalVector {
- get { return locals; }
- }
-
//
// Debugging stuff.
//
public override string ToString ()
{
- StringBuilder sb = new StringBuilder ();
-
- sb.Append ("Vector (");
- sb.Append (Type);
- sb.Append (",");
- sb.Append (id);
- sb.Append (",");
- sb.Append (IsDirty);
- sb.Append (",");
- sb.Append (reachability);
- if (parameters != null) {
- sb.Append (" - ");
- sb.Append (parameters);
- }
- sb.Append (" - ");
- sb.Append (locals);
- sb.Append (")");
-
- return sb.ToString ();
+ return String.Format ("Vector ({0},{1},{2}-{3}-{4})", Type, id, reachability, parameters, locals);
}
}
protected abstract void AddSibling (UsageVector uv);
- public virtual LabeledStatement LookupLabel (string name, Location loc)
- {
- if (Parent != null)
- return Parent.LookupLabel (name, loc);
-
- Report.Error (
- 159, loc,
- "No such label `" + name + "' in this scope");
- return null;
- }
-
- public abstract void Label (UsageVector origin_vectors);
-
- // <summary>
- // Check whether all `out' parameters have been assigned.
- // </summary>
- public void CheckOutParameters (MyBitVector parameters, Location loc)
- {
- if (parameters == null)
- return;
-
- for (int i = 0; i < param_map.Count; i++) {
- VariableInfo var = param_map [i];
-
- if (var == null)
- continue;
-
- if (var.IsAssigned (parameters))
- continue;
-
- Report.Error (177, loc, "The out parameter `{0}' must be assigned to before control leaves the current method",
- var.Name);
- }
- }
-
- protected UsageVector Merge (UsageVector sibling_list)
- {
- if (sibling_list.Next == null)
- return sibling_list;
-
- MyBitVector locals = null;
- MyBitVector parameters = null;
-
- Reachability reachability = null;
-
- Report.Debug (2, " MERGING SIBLINGS", this, Name);
-
- for (UsageVector child = sibling_list; child != null; child = child.Next) {
- Report.Debug (2, " MERGING SIBLING ", reachability, child);
-
- if (reachability == null)
- reachability = child.Reachability.Clone ();
- else
- reachability.Meet (child.Reachability);
-
- // A local variable is initialized after a flow branching if it
- // has been initialized in all its branches which do neither
- // always return or always throw an exception.
- //
- // If a branch may return, but does not always return, then we
- // can treat it like a never-returning branch here: control will
- // only reach the code position after the branching if we did not
- // return here.
- //
- // It's important to distinguish between always and sometimes
- // returning branches here:
- //
- // 1 int a;
- // 2 if (something) {
- // 3 return;
- // 4 a = 5;
- // 5 }
- // 6 Console.WriteLine (a);
- //
- // The if block in lines 3-4 always returns, so we must not look
- // at the initialization of `a' in line 4 - thus it'll still be
- // uninitialized in line 6.
- //
- // On the other hand, the following is allowed:
- //
- // 1 int a;
- // 2 if (something)
- // 3 a = 5;
- // 4 else
- // 5 return;
- // 6 Console.WriteLine (a);
- //
- // Here, `a' is initialized in line 3 and we must not look at
- // line 5 since it always returns.
- //
- bool unreachable = child.Reachability.IsUnreachable;
-
- Report.Debug (2, " MERGING SIBLING #1", reachability,
- Type, child.Type, child.Reachability.IsUnreachable, unreachable);
-
- if (!unreachable && (child.LocalVector != null))
- MyBitVector.And (ref locals, child.LocalVector);
-
- // An `out' parameter must be assigned in all branches which do
- // not always throw an exception.
- if ((child.ParameterVector != null) && !child.Reachability.AlwaysThrows)
- MyBitVector.And (ref parameters, child.ParameterVector);
-
- Report.Debug (2, " MERGING SIBLING #2", parameters, locals);
- }
-
- if (reachability == null)
- throw new InternalErrorException ("Cannot happen: the loop above runs at least twice");
-
- Report.Debug (2, " MERGING SIBLINGS DONE", parameters, locals, reachability);
-
- return new UsageVector (parameters, locals, reachability, null, Location);
- }
-
protected abstract UsageVector Merge ();
// <summary>
// </summary>
public UsageVector MergeChild (FlowBranching child)
{
- bool implicit_block = child.Type == BranchingType.Block && child.Block.Implicit;
+ bool overwrite = child.Type == BranchingType.Labeled ||
+ (child.Type == BranchingType.Block && child.Block.Implicit);
Report.Debug (2, " MERGING CHILD", this, child);
- UsageVector result = CurrentUsageVector.MergeChild (child.Merge (), implicit_block);
+ UsageVector result = CurrentUsageVector.MergeChild (child.Merge (), overwrite);
Report.Debug (2, " MERGING CHILD DONE", this, result);
return result;
}
- // <summary>
- // Does the toplevel merging.
- // </summary>
- public Reachability MergeTopBlock ()
- {
- if ((Type != BranchingType.Block) || (Block == null))
- throw new NotSupportedException ();
-
- UsageVector result = Merge ();
-
- Report.Debug (4, "MERGE TOP BLOCK", Location, result);
-
- if (!result.Reachability.AlwaysThrows && !result.Reachability.AlwaysHasBarrier)
- CheckOutParameters (result.Parameters, Location);
-
- return result.Reachability;
- }
-
public virtual bool InTryWithCatch ()
{
- return Parent != null && Parent.InTryWithCatch ();
+ return Parent.InTryWithCatch ();
}
// returns true if we crossed an unwind-protected region (try/catch/finally, lock, using, ...)
public virtual bool AddBreakOrigin (UsageVector vector, Location loc)
{
- if (Parent != null)
- return Parent.AddBreakOrigin (vector, loc);
-
- Report.Error (139, loc, "No enclosing loop out of which to break or continue");
- return false;
+ return Parent.AddBreakOrigin (vector, loc);
}
// returns true if we crossed an unwind-protected region (try/catch/finally, lock, using, ...)
public virtual bool AddContinueOrigin (UsageVector vector, Location loc)
{
- if (Parent != null)
- return Parent.AddContinueOrigin (vector, loc);
-
- Report.Error (139, loc, "No enclosing loop out of which to break or continue");
- return false;
+ return Parent.AddContinueOrigin (vector, loc);
}
// returns true if we crossed an unwind-protected region (try/catch/finally, lock, using, ...)
public virtual bool AddReturnOrigin (UsageVector vector, Location loc)
{
- if (Parent != null)
- return Parent.AddReturnOrigin (vector, loc);
+ return Parent.AddReturnOrigin (vector, loc);
+ }
- CheckOutParameters (vector.Parameters, loc);
- return false;
+ // returns true if we crossed an unwind-protected region (try/catch/finally, lock, using, ...)
+ public virtual bool AddGotoOrigin (UsageVector vector, Goto goto_stmt)
+ {
+ return Parent.AddGotoOrigin (vector, goto_stmt);
}
public virtual void StealFinallyClauses (ref ArrayList list)
{
- if (Parent != null)
- Parent.StealFinallyClauses (ref list);
+ Parent.StealFinallyClauses (ref list);
}
public bool IsAssigned (VariableInfo vi)
sibling_list = sibling;
}
- public override LabeledStatement LookupLabel (string name, Location loc)
+ public override bool AddGotoOrigin (UsageVector vector, Goto goto_stmt)
{
- if (Block == null)
- return base.LookupLabel (name, loc);
-
- LabeledStatement s = Block.LookupLabel (name);
- if (s != null)
- return s;
+ LabeledStatement stmt = Block == null ? null : Block.LookupLabel (goto_stmt.Target);
+ if (stmt == null)
+ return Parent.AddGotoOrigin (vector, goto_stmt);
- return base.LookupLabel (name, loc);
- }
-
- public override void Label (UsageVector origin_vectors)
- {
- if (!CurrentUsageVector.Reachability.IsUnreachable) {
- UsageVector vector = CurrentUsageVector.Clone ();
- vector.Next = origin_vectors;
- origin_vectors = vector;
- }
-
- CurrentUsageVector.MergeJumpOrigins (origin_vectors);
+ // forward jump
+ goto_stmt.SetResolvedTarget (stmt);
+ stmt.AddUsageVector (vector);
+ return false;
}
protected override UsageVector Merge ()
{
- return Merge (sibling_list);
+ Report.Debug (2, " MERGING SIBLINGS", Name);
+ UsageVector vector = UsageVector.MergeSiblings (sibling_list, Location);
+ Report.Debug (2, " MERGING SIBLINGS DONE", Name, vector);
+ return vector;
}
}
public class FlowBranchingLabeled : FlowBranchingBlock
{
LabeledStatement stmt;
+ UsageVector actual;
+
public FlowBranchingLabeled (FlowBranching parent, LabeledStatement stmt)
: base (parent, BranchingType.Labeled, SiblingType.Conditional, null, stmt.loc)
{
this.stmt = stmt;
+ CurrentUsageVector.MergeOrigins (stmt.JumpOrigins);
+ actual = CurrentUsageVector.Clone ();
+
+ // stand-in for backward jumps
+ CurrentUsageVector.Reachability.Meet (Reachability.Always ());
+ }
+
+ public override bool AddGotoOrigin (UsageVector vector, Goto goto_stmt)
+ {
+ if (goto_stmt.Target != stmt.Name)
+ return Parent.AddGotoOrigin (vector, goto_stmt);
+
+ // backward jump
+ goto_stmt.SetResolvedTarget (stmt);
+ actual.MergeOrigins (vector.Clone ());
+
+ return false;
+ }
+
+ protected override UsageVector Merge ()
+ {
+ UsageVector vector = base.Merge ();
+
+ if (actual.Reachability.IsUnreachable)
+ Report.Warning (162, 2, stmt.loc, "Unreachable code detected");
+
+ actual.MergeChild (vector, false);
+ return actual;
+ }
+ }
+
+ public class FlowBranchingToplevel : FlowBranchingBlock
+ {
+ UsageVector return_origins;
+
+ public FlowBranchingToplevel (FlowBranching parent, ToplevelBlock stmt)
+ : base (parent, BranchingType.Toplevel, SiblingType.Conditional, stmt, stmt.loc)
+ {
+ }
+
+ // <summary>
+ // Check whether all `out' parameters have been assigned.
+ // </summary>
+ void CheckOutParameters (UsageVector vector, Location loc)
+ {
+ if (vector.Reachability.IsUnreachable)
+ return;
+ for (int i = 0; i < param_map.Count; i++) {
+ VariableInfo var = param_map [i];
+
+ if (var == null)
+ continue;
+
+ if (vector.IsAssigned (var, false))
+ continue;
+
+ Report.Error (177, loc, "The out parameter `{0}' must be assigned to before control leaves the current method",
+ var.Name);
+ }
+ }
+
+ public override bool InTryWithCatch ()
+ {
+ return false;
+ }
+
+ public override bool AddBreakOrigin (UsageVector vector, Location loc)
+ {
+ Report.Error (139, loc, "No enclosing loop out of which to break or continue");
+ return false;
+ }
+
+ public override bool AddContinueOrigin (UsageVector vector, Location loc)
+ {
+ Report.Error (139, loc, "No enclosing loop out of which to break or continue");
+ return false;
+ }
+
+ public override bool AddReturnOrigin (UsageVector vector, Location loc)
+ {
+ vector = vector.Clone ();
+ vector.Location = loc;
+ vector.Next = return_origins;
+ return_origins = vector;
+ return false;
+ }
+
+ public override void StealFinallyClauses (ref ArrayList list)
+ {
+ // nothing to do
+ }
+
+ public override bool AddGotoOrigin (UsageVector vector, Goto goto_stmt)
+ {
+ string name = goto_stmt.Target;
+ LabeledStatement s = Block.LookupLabel (name);
+ if (s != null)
+ throw new InternalErrorException ("Shouldn't get here");
+
+ if (Parent == null) {
+ Report.Error (159, goto_stmt.loc, "No such label `{0}' in this scope", name);
+ return false;
+ }
+
+ int errors = Report.Errors;
+ Parent.AddGotoOrigin (vector, goto_stmt);
+ if (errors == Report.Errors)
+ Report.Error (1632, goto_stmt.loc, "Control cannot leave the body of an anonymous method");
+ return false;
+ }
+
+ protected override UsageVector Merge ()
+ {
+ for (UsageVector origin = return_origins; origin != null; origin = origin.Next)
+ CheckOutParameters (origin, origin.Location);
+
+ UsageVector vector = base.Merge ();
+ CheckOutParameters (vector, Block.loc);
+ // Note: we _do_not_ merge in the return origins
+ return vector;
+ }
+
+ public Reachability End ()
+ {
+ return Merge ().Reachability;
}
}
UsageVector break_origins;
UsageVector continue_origins;
UsageVector return_origins;
+ GotoOrigin goto_origins;
+
+ class GotoOrigin {
+ public GotoOrigin Next;
+ public Goto GotoStmt;
+ public UsageVector Vector;
+
+ public GotoOrigin (UsageVector vector, Goto goto_stmt, GotoOrigin next)
+ {
+ Vector = vector;
+ GotoStmt = goto_stmt;
+ Next = next;
+ }
+ }
bool emit_finally;
public override bool AddBreakOrigin (UsageVector vector, Location loc)
{
+ vector = vector.Clone ();
if (finally_vector != null) {
- Report.Error (157, loc, "Control cannot leave the body of a finally clause");
+ vector.MergeChild (finally_vector, false);
+ int errors = Report.Errors;
+ Parent.AddBreakOrigin (vector, loc);
+ if (errors == Report.Errors)
+ Report.Error (157, loc, "Control cannot leave the body of a finally clause");
} else {
- vector = vector.Clone ();
vector.Location = loc;
vector.Next = break_origins;
break_origins = vector;
public override bool AddContinueOrigin (UsageVector vector, Location loc)
{
+ vector = vector.Clone ();
if (finally_vector != null) {
- Report.Error (157, loc, "Control cannot leave the body of a finally clause");
+ vector.MergeChild (finally_vector, false);
+ int errors = Report.Errors;
+ Parent.AddContinueOrigin (vector, loc);
+ if (errors == Report.Errors)
+ Report.Error (157, loc, "Control cannot leave the body of a finally clause");
} else {
- vector = vector.Clone ();
vector.Location = loc;
vector.Next = continue_origins;
continue_origins = vector;
public override bool AddReturnOrigin (UsageVector vector, Location loc)
{
+ vector = vector.Clone ();
if (finally_vector != null) {
- Report.Error (157, loc, "Control cannot leave the body of a finally clause");
+ vector.MergeChild (finally_vector, false);
+ int errors = Report.Errors;
+ Parent.AddReturnOrigin (vector, loc);
+ if (errors == Report.Errors)
+ Report.Error (157, loc, "Control cannot leave the body of a finally clause");
} else {
- vector = vector.Clone ();
vector.Location = loc;
vector.Next = return_origins;
return_origins = vector;
return true;
}
+ public override bool AddGotoOrigin (UsageVector vector, Goto goto_stmt)
+ {
+ LabeledStatement s = current_vector.Block == null ? null : current_vector.Block.LookupLabel (goto_stmt.Target);
+ if (s != null)
+ throw new InternalErrorException ("Shouldn't get here");
+
+ vector = vector.Clone ();
+ if (finally_vector != null) {
+ vector.MergeChild (finally_vector, false);
+ int errors = Report.Errors;
+ Parent.AddGotoOrigin (vector, goto_stmt);
+ if (errors == Report.Errors)
+ Report.Error (157, goto_stmt.loc, "Control cannot leave the body of a finally clause");
+ } else {
+ goto_origins = new GotoOrigin (vector, goto_stmt, goto_origins);
+ }
+ return true;
+ }
+
public override void StealFinallyClauses (ref ArrayList list)
{
if (list == null)
get { return emit_finally; }
}
- public override LabeledStatement LookupLabel (string name, Location loc)
- {
- if (current_vector.Block == null)
- return base.LookupLabel (name, loc);
-
- LabeledStatement s = current_vector.Block.LookupLabel (name);
- if (s != null)
- return s;
-
- if (finally_vector != null) {
- Report.Error (157, loc,
- "Control cannot leave the body of a finally clause");
- return null;
- }
-
- return base.LookupLabel (name, loc);
- }
-
- public override void Label (UsageVector origin_vectors)
- {
- CurrentUsageVector.MergeJumpOrigins (origin_vectors);
- }
-
protected override UsageVector Merge ()
{
- UsageVector vector = Merge (catch_vectors);
+ Report.Debug (2, " MERGING TRY/CATCH", Name);
+ UsageVector vector = UsageVector.MergeSiblings (catch_vectors, Location);
+ Report.Debug (2, " MERGING TRY/CATCH DONE", vector);
if (finally_vector != null)
vector.MergeChild (finally_vector, false);
for (UsageVector origin = break_origins; origin != null; origin = origin.Next) {
if (finally_vector != null)
origin.MergeChild (finally_vector, false);
- if (!origin.Reachability.IsUnreachable)
- Parent.AddBreakOrigin (origin, origin.Location);
+ Parent.AddBreakOrigin (origin, origin.Location);
}
for (UsageVector origin = continue_origins; origin != null; origin = origin.Next) {
if (finally_vector != null)
origin.MergeChild (finally_vector, false);
- if (!origin.Reachability.IsUnreachable)
- Parent.AddContinueOrigin (origin, origin.Location);
+ Parent.AddContinueOrigin (origin, origin.Location);
}
for (UsageVector origin = return_origins; origin != null; origin = origin.Next) {
if (finally_vector != null)
origin.MergeChild (finally_vector, false);
- if (!origin.Reachability.IsUnreachable)
- Parent.AddReturnOrigin (origin, origin.Location);
+ Parent.AddReturnOrigin (origin, origin.Location);
+ }
+
+ for (GotoOrigin origin = goto_origins; origin != null; origin = origin.Next) {
+ if (finally_vector != null)
+ origin.Vector.MergeChild (finally_vector, false);
+ Parent.AddGotoOrigin (origin.Vector, origin.GotoStmt);
}
return vector;
ArrayList non_public_fields = new ArrayList ();
if (fields != null) {
- foreach (FieldMember field in fields) {
+ foreach (FieldBase field in fields) {
if ((field.ModFlags & Modifiers.STATIC) != 0)
continue;
if ((field.ModFlags & Modifiers.PUBLIC) != 0)
Fields = new FieldInfo [Count];
public_fields.CopyTo (Fields, 0);
non_public_fields.CopyTo (Fields, CountPublic);
+#if GMCS_SOURCE
+ } else if (type is GenericTypeParameterBuilder) {
+ CountPublic = CountNonPublic = Count = 0;
+
+ Fields = new FieldInfo [0];
+#endif
} else {
FieldInfo[] public_fields = type.GetFields (
BindingFlags.Instance|BindingFlags.Public);
public bool IsAssigned (MyBitVector vector)
{
+ if (vector == null)
+ return true;
+
if (vector [Offset])
return true;
// </summary>
public class MyBitVector {
public readonly int Count;
- public readonly MyBitVector InheritsFrom;
+ public static readonly MyBitVector Empty = new MyBitVector ();
- bool is_dirty;
- BitArray vector;
+ // Invariant: vector != null => vector.Count == Count
+ // Invariant: vector == null || shared == null
+ // i.e., at most one of 'vector' and 'shared' can be non-null. They can both be null -- that means all-ones
+ // The object in 'shared' cannot be modified, while 'vector' can be freely modified
+ BitArray vector, shared;
- public MyBitVector (int Count)
- : this (null, Count)
- { }
+ MyBitVector ()
+ {
+ shared = new BitArray (0, false);
+ }
public MyBitVector (MyBitVector InheritsFrom, int Count)
{
- this.InheritsFrom = InheritsFrom;
+ if (InheritsFrom != null)
+ shared = InheritsFrom.Shared;
+
this.Count = Count;
}
- // <summary>
- // Checks whether this bit vector has been modified. After setting this to true,
- // we won't use the inherited vector anymore, but our own copy of it.
- // </summary>
- public bool IsDirty {
- get { return is_dirty; }
-
- set {
- if (!is_dirty)
- initialize_vector ();
+ // Use this accessor to get a shareable copy of the underlying BitArray representation
+ BitArray Shared {
+ get {
+ // Post-condition: vector == null
+ if (shared == null) {
+ shared = vector;
+ vector = null;
+ }
+ return shared;
}
}
// <summary>
// Get/set bit `index' in the bit vector.
// </summary>
- public bool this [int index]
- {
+ public bool this [int index] {
get {
- if (index > Count)
+ if (index >= Count)
throw new ArgumentOutOfRangeException ();
- // We're doing a "copy-on-write" strategy here; as long
- // as nobody writes to the array, we can use our parent's
- // copy instead of duplicating the vector.
-
if (vector != null)
return vector [index];
- else if (InheritsFrom != null) {
- BitArray inherited = InheritsFrom.Vector;
-
- if (index < inherited.Count)
- return inherited [index];
- else
- return false;
- } else
- return false;
+ if (shared == null)
+ return true;
+ if (index < shared.Count)
+ return shared [index];
+ return false;
}
set {
- if (index > Count)
- throw new ArgumentOutOfRangeException ();
-
// Only copy the vector if we're actually modifying it.
-
if (this [index] != value) {
- initialize_vector ();
-
+ if (vector == null)
+ initialize_vector ();
vector [index] = value;
}
}
}
- // <summary>
- // If you explicitly convert the MyBitVector to a BitArray, you will get a deep
- // copy of the bit vector.
- // </summary>
- public static explicit operator BitArray (MyBitVector vector)
- {
- vector.initialize_vector ();
- return vector.Vector;
- }
-
// <summary>
// Performs an `or' operation on the bit vector. The `new_vector' may have a
// different size than the current one.
// </summary>
- public void Or (MyBitVector new_vector)
+ private MyBitVector Or (MyBitVector new_vector)
{
- // Treat null 'new_vector' as all false, just like the And() below
- if (new_vector == null)
- return;
- BitArray new_array = new_vector.Vector;
+ if (Count == 0 || new_vector.Count == 0)
+ return this;
- initialize_vector ();
+ BitArray o = new_vector.vector != null ? new_vector.vector : new_vector.shared;
- int upper;
- if (vector.Count < new_array.Count)
- upper = vector.Count;
- else
- upper = new_array.Count;
+ if (o == null) {
+ int n = new_vector.Count;
+ if (n < Count) {
+ for (int i = 0; i < n; ++i)
+ this [i] = true;
+ } else {
+ SetAll (true);
+ }
+ return this;
+ }
- for (int i = 0; i < upper; i++)
- vector [i] = vector [i] | new_array [i];
+ if (Count == o.Count) {
+ if (vector == null) {
+ if (shared == null)
+ return this;
+ initialize_vector ();
+ }
+ vector.Or (o);
+ return this;
+ }
+
+ int min = o.Count;
+ if (Count < min)
+ min = Count;
+
+ for (int i = 0; i < min; i++) {
+ if (o [i])
+ this [i] = true;
+ }
+
+ return this;
}
// <summary>
- // Perfonrms an `and' operation on the bit vector. The `new_vector' may have
+ // Performs an `and' operation on the bit vector. The `new_vector' may have
// a different size than the current one.
// </summary>
- public void And (MyBitVector new_vector)
+ private MyBitVector And (MyBitVector new_vector)
{
- BitArray new_array;
+ if (Count == 0)
+ return this;
- if (new_vector != null)
- new_array = new_vector.Vector;
- else
- new_array = new BitArray (Count, false);
+ BitArray o = new_vector.vector != null ? new_vector.vector : new_vector.shared;
- initialize_vector ();
+ if (o == null) {
+ for (int i = new_vector.Count; i < Count; ++i)
+ this [i] = false;
+ return this;
+ }
- int lower, upper;
- if (vector.Count < new_array.Count)
- lower = upper = vector.Count;
- else {
- lower = new_array.Count;
- upper = vector.Count;
+ if (o.Count == 0) {
+ SetAll (false);
+ return this;
}
- for (int i = 0; i < lower; i++)
- vector [i] = vector [i] & new_array [i];
+ if (Count == o.Count) {
+ if (vector == null) {
+ if (shared == null) {
+ shared = new_vector.Shared;
+ return this;
+ }
+ initialize_vector ();
+ }
+ vector.And (o);
+ return this;
+ }
+
+ int min = o.Count;
+ if (Count < min)
+ min = Count;
+
+ for (int i = 0; i < min; i++) {
+ if (! o [i])
+ this [i] = false;
+ }
+
+ for (int i = min; i < Count; i++)
+ this [i] = false;
- for (int i = lower; i < upper; i++)
- vector [i] = false;
+ return this;
}
- public static void And (ref MyBitVector target, MyBitVector vector)
+ public static MyBitVector operator & (MyBitVector a, MyBitVector b)
{
- if (target != null)
- target.And (vector);
+ if (a == b)
+ return a;
+ if (a == null)
+ return b.Clone ();
+ if (b == null)
+ return a.Clone ();
+ if (a.Count > b.Count)
+ return a.Clone ().And (b);
else
- target = vector.Clone ();
+ return b.Clone ().And (a);
}
- public static void Or (ref MyBitVector target, MyBitVector vector)
+ public static MyBitVector operator | (MyBitVector a, MyBitVector b)
{
- if (target != null)
- target.Or (vector);
+ if (a == b)
+ return a;
+ if (a == null)
+ return new MyBitVector (null, b.Count);
+ if (b == null)
+ return new MyBitVector (null, a.Count);
+ if (a.Count > b.Count)
+ return a.Clone ().Or (b);
else
- target = vector.Clone ();
+ return b.Clone ().Or (a);
}
- // <summary>
- // This does a deep copy of the bit vector.
- // </summary>
public MyBitVector Clone ()
{
- MyBitVector retval = new MyBitVector (Count);
-
- retval.Vector = Vector;
-
- return retval;
+ return Count == 0 ? Empty : new MyBitVector (this, Count);
}
- BitArray Vector {
- get {
- if (vector != null)
- return vector;
- else if (!is_dirty && (InheritsFrom != null))
- return InheritsFrom.Vector;
-
- initialize_vector ();
-
- return vector;
- }
-
- set {
- initialize_vector ();
-
- for (int i = 0; i < System.Math.Min (vector.Count, value.Count); i++)
- vector [i] = value [i];
- }
+ public void SetAll (bool value)
+ {
+ // Don't clobber Empty
+ if (Count == 0)
+ return;
+ shared = value ? null : Empty.Shared;
+ vector = null;
}
void initialize_vector ()
{
- if (vector != null)
+ // Post-condition: vector != null
+ if (shared == null) {
+ vector = new BitArray (Count, true);
return;
-
- vector = new BitArray (Count, false);
- if (InheritsFrom != null)
- Vector = InheritsFrom.Vector;
+ }
- is_dirty = true;
+ vector = new BitArray (shared);
+ if (Count != vector.Count)
+ vector.Length = Count;
+ shared = null;
}
- public override string ToString ()
+ StringBuilder Dump (StringBuilder sb)
{
- StringBuilder sb = new StringBuilder ("{");
-
- BitArray vector = Vector;
- if (!IsDirty)
+ BitArray dump = vector == null ? shared : vector;
+ if (dump == null)
+ return sb.Append ("/");
+ if (dump == shared)
sb.Append ("=");
- for (int i = 0; i < vector.Count; i++) {
- sb.Append (vector [i] ? "1" : "0");
- }
-
- sb.Append ("}");
- return sb.ToString ();
+ for (int i = 0; i < dump.Count; i++)
+ sb.Append (dump [i] ? "1" : "0");
+ return sb;
+ }
+
+ public override string ToString ()
+ {
+ return Dump (new StringBuilder ("{")).Append ("}").ToString ();
}
}
}