| Last change
 on this file since 47b7142 was             f02f546, checked in by Michael Brooks <mlbrooks@…>, 2 years ago | 
        
          | 
Implement new rules for array dimension expression matching.
 
Core changes:src/ResolvExpr/Unify.cc: add sense of "these two expressions unify"
 src/InitTweak/GenInit.cc: make hoisting happen more often
 tests/array-container/*: reconfigure the array-dimension test to use non-"classic" expectation rules
 
 
Misc contributors and noise:libcfa/src/parseargs.cfa: ,ake a parameter, that's used as a length expression, constant (example of an array user following new rules)
 src/ResolvExpr/ResolveTypeof.h: make fixArrayType public
 src/Validate/GenericParameter.cpp: do the array-type desugaring in both AST corners that forall-variables can be found (not just in one of them)
 tests/.expect/typedefRedef-ERR1.txt: old one was "expecting" a bug, that new array rules handle correctly
 
 | 
        
          | 
              
Property                 mode
 set to                 100644 | 
        
          | File size:
            1.4 KB | 
      
      
| Line |  | 
|---|
| 1 | // A white-box case. | 
|---|
| 2 |  | 
|---|
| 3 | // Array dimension hoisting (gen-init, just before resolver) is the earliest pass that uses a | 
|---|
| 4 | // with-identifiers symbol table, so early that it's premature to have the symbol table report | 
|---|
| 5 | // on name collisions/inconsistencies, and so the symbol table runs in a mode that ignores them. | 
|---|
| 6 |  | 
|---|
| 7 | // This test exercises a dimention-hoisting symbol-table query that retrieves an inconsistently | 
|---|
| 8 | // defined identifier.  This fact can be verified manually in gdb by observing a call to | 
|---|
| 9 | // SymbolTable.cpp's OnFindError (during an addId call with key `collide_me`) that returns | 
|---|
| 10 | // without throwing an exception, followed by a lookupId call with key `collide_me`. | 
|---|
| 11 |  | 
|---|
| 12 | // The (black-box obvious) expected outcome is that cfa-cc rejects the input program on account | 
|---|
| 13 | // of the inconsistent identifier. | 
|---|
| 14 |  | 
|---|
| 15 | // The (white-box) relevance of achieving this outcome is the demonstration that an early ignorred | 
|---|
| 16 | // name collision leaves the compiler in a valid-enough state to reach the later collision | 
|---|
| 17 | // detection, without implementation shotcuts like, "Which, if either, declaration gets | 
|---|
| 18 | // ignorred?" becoming relevant. | 
|---|
| 19 |  | 
|---|
| 20 | // This test is unrelated to _why_ the dimension-hoising pass needs to disable symbol-table | 
|---|
| 21 | // validation; linking/manging/anon is such a case. | 
|---|
| 22 |  | 
|---|
| 23 | extern "C" { | 
|---|
| 24 | const long long unsigned collide_me = 42; | 
|---|
| 25 | const long      unsigned collide_me = 17; | 
|---|
| 26 | } | 
|---|
| 27 |  | 
|---|
| 28 | int main() { | 
|---|
| 29 | float a[collide_me]; | 
|---|
| 30 | } | 
|---|
       
      
  Note:
 See   
TracBrowser
 for help on using the repository browser.