-Major tasks:
-------------
-
- Apply the method verification process to indexers and properties as well.
-
- Properties and 17.6.3: Finish it.
+TODO:
- Finish pre-processor (and, or and nested expressions not yet supported).
+ 1. Create a "partial" emit context for each TypeContainer..
- Implement base indexer access.
-
- base property access on a virtual method.
-
- Final should not be set for abstract method implementations.
- final should be set for implementations of interface methods.
+ 2. EmitContext should be partially constructed. No IL Generator.
-BUGS
-----
+ Optimize the internal type expressions in the parser (system_object, etc)
+ to be something other than DecomposeQI-based expressions.
-* Major bug:
+ interface_type review.
- Since I added the support for the dual interpreatation of names, I dont
- catch this error:
+ parameter_array, line 952: `note: must be a single dimension array type'. Validate this
- class X { int i; static void a () { i = 1; } }
+Dead Code Elimination bugs:
+---------------------------
-* Currently the code path for 108/109 reporting is not being ran for methods
- as we need to compare method signatures. But since we retrieve the expensive
- method arguments in the method, we probably should do 108/109 processing there.
+ I should also resolve all the children expressions in Switch, Fixed, Using.
-* Emit warning on hiding members without NEW not only in members.
-
-* Implement visibility.
-
-* Adding variables.
-
- We do add variables in a number of places, and this is erroneous:
-
- void a (int b)
- {
- int b;
- }
+Major tasks:
+------------
- Also:
+ Pinned and volatile require type modifiers that can not be encoded
+ with Reflection.Emit.
- void a (int b)
- {
- foreach (int b ...)
- ;
- }
+ Properties and 17.6.3: Finish it.
-* Visibility
+ Implement base indexer access.
- I am not reporting errors on visibility yet.
+readonly variables and ref/out
+
+BUGS
+----
-* Interfaces
+* We suck at reporting what turns out to be error -6. Use the standard error message
+ instead.
- For indexers, the output of ix2.cs is different from our
- compiler and theirs. They use a DefaultMemberAttribute, which
- I have yet to figure out:
+* Explicit indexer implementation is missing.
- .class interface private abstract auto ansi INTERFACE
- {
- .custom instance void [mscorlib]System.Reflection.DefaultMemberAttribute::.ctor(string)
- = ( 01 00 04 49 74 65 6D 00 00 ) // ...Item..
- ...
- }
+* Check for Final when overriding, if the parent is Final, then we cant
+ allow an override.
* Interface indexers
Explanation: The reason for the `instance' attribute on
indexers is that indexers only apply to instances
-* Handle destructors specially
-
- Turn ~X () { a () } into:
- void Finalize () { try { a (); } finally { base.Finalize (); } }
-
- The code is mostly done, but `base' is missing. The reason it is
- missing is because we have to implement visibility first.
-
-* Arrays
-
- We need to make sure at *compile time* that the arguments in
- the expression list of an array creation are always positive.
-
-* Implement dead code elimination in statement.cs
-
- It is pretty simple to implement dead code elimination in
- if/do/while
-
* Indexer bugs:
the following wont work:
They should transfer control to the finally block if inside a try/catch
block.
-* Conversions and overflow
-
- I am not using the checked state while doing type casts,
- they should result in conv.ovf.XXX
-
* Method Registration and error CS111
The way we use the method registration to signal 111 is wrong.
PENDING TASKS
-------------
- * Implement Goto.
+* IMprove error handling here:
- * Unsafe code.
- Making the parser accept unsafe code.
-
- * Implement `base' (BaseAccess class)
+ public static Process ()
- * Implement constant folding. We might need to implement
- a new `Constant' class and derive Literal from it.
+ THat assumes that it is a constructor, check if its the same name
+ as the class, if not report a different error than the one we use now.
-* OUT Variables passed as OUT variables.
+* Merge test 89 and test-34
- Currently we pass the pointer to the pointer (see expression.cs:Argument.Emit)
-
-* Make the rules for inheritance/overriding apply to
- properties, operators and indexers (currently we only do this
- on methods).
-
-* Handle volatile
+* Revisit
+ Primary-expression, as it has now been split into
+ non-array-creation-expression and array-creation-expression.
+
* Static flow analysis
Required to warn about reachability of code and definite
The information when registering a method in InternalParameters
is duplicated, you can always get the types from the InternalParameters
+* Emit modreq for volatiles
+
+ Handle modreq from public apis.
+
+* Emit `pinned' for pinned local variables.
+
+ Both `modreq' and pinned will require special hacks in the compiler.
+
+* Make sure that we are pinning the right variable
+
+* Maybe track event usage? Currently I am not tracking these, although they
+ are fields.
+
+* Merge tree.cs, rootcontext.cs
+
OPTIMIZATIONS
-------------
-* Emitcontext
+* Implement loop inversion for `For' as well.
- Do we really need to instanciate this variable all the time?
+* There is too much unshared code between MemberAccess.Resolve and SimpleName
+ resolve.
+
+* User Defined Conversions is doing way too many calls to do union sets that are not needed
- It could be static for all we care, and just use it for making
- sure that there are no recursive invocations on it.
+* Add test case for destructors
+
+* Places that use `Ldelema' are basically places where I will be
+ initializing a value type. I could apply an optimization to
+ disable the implicit local temporary from being created (by using
+ the method in New).
+
+* Dropping TypeContainer as an argument to EmitContext
-* Static-ization
+ My theory is that I can get rid of the TypeBuilder completely from
+ the EmitContext, and have typecasts where it is used (from
+ DeclSpace to where it matters).
- Since AppDomain exists, maybe we can get rid of all the stuff
- that is part of the `compiler instance' and just use globals
- everywhere.
+ The only pending problem is that the code that implements Aliases
+ is on TypeContainer, and probably should go in DeclSpace.
+* Casts need to trigger a name resolution against types only.
-* Constructors
+ currently we use a secret hand shake, probably we should use
+ a differen path, and only expressions (memberaccess, simplename)
+ would participate in this protocol.
- Currently it calls the parent constructor before initializing fields.
- It should do it the other way around.
+* Use of local temporary in UnaryMutator
+
+ We should get rid of the Localtemporary there for some cases
+
+* Emitcontext
+
+ Do we really need to instanciate this variable all the time?
+
+ It could be static for all we care, and just use it for making
+ sure that there are no recursive invocations on it.
* Use of EmitBranchable
Write tests for the various reference conversions. We have
test for all the numeric conversions.
-* Remove the tree dumper, cleanup `public readonly'
-
- And make all the stuff which is `public readonly' be private unless
- required.
-
* Optimizations
In Indexers and Properties, probably support an EmitWithDup
in the stack, so that later a Store can be emitted using that
this pointer (consider Property++ or Indexer++)
-
* Optimizations: variable allocation.
When local variables of a type are required, we should request
* Add a cache for the various GetArrayMethod operations.
-* Optimization:
-
- Do not use StatementCollections, use ArrayLists of Statements,
- and then "copy" it out to arrays. Then reuse the existing
- Statement structures.
-
* TypeManager.FindMembers:
Instead of having hundreds of builder_to_blah hash table, have
a single one that maps a TypeBuilder `t' to a set of classes
that implement an interface that supports FindMembers.
+* MakeUnionSet Callers
+
+ If the types are the same, there is no need to compute the unionset,
+ we can just use the list from one of the types.
+
+* Factor all the FindMembers in all the FindMembers providers.
+
+* Factor the lookup code for class declarations an interfaces
+ (interface.cs:GetInterfaceByName)
+
RECOMMENDATIONS
---------------
Not sure that this grammar is correct, we might have to
resolve this during semantic analysis.
+* Idea
-* Try/Catch
+ MethodGroupExpr
- Investigate what is the right value to return from `Emit' in
- there (ie, for the `all code paths return')
-
-
-* Optimizations
+ These guys should only appear as part of an Invocation, so we
+ probably can afford to have a special callback:
- Only create one `This' instance per class, and reuse it.
-
- Maybe keep a pool of constants/literals (zero, 1)?
-
-************
-Potential bug:
-
- We would need to decode the shortname before we lookup members?
-
- Maybe not.
+ Expression.ResolveAllowMemberGroups
-interface I {
- void A ();
-}
+ This is only called by Invocation (or anyone that consumes
+ MethodGroupExprs)
-class X : I {
- void I.A ();
-}
+ And the regular DoResolve and DoResolveLValue do emit the error
+ 654 `Method referenced without argument list'.
-class Y : X, I {
- void I.A () {}
-}
\ No newline at end of file
+ Otherwise, a resolution will return a MethodGroupExpr which is
+ not guaranteed to have set its `Expression.Type' to a non-null
+ value.