Skip to content
Permalink

Comparing changes

Choose two branches to see what’s changed or to start a new pull request. If you need to, you can also or learn more about diff comparisons.

Open a pull request

Create a new pull request by comparing changes across two branches. If you need to, you can also . Learn more about diff comparisons here.
base repository: m-labs/flickernoise
base: d755a0b
Choose a base ref
...
head repository: m-labs/flickernoise
compare: 6259a03
Choose a head ref
  • 10 commits
  • 27 files changed
  • 1 contributor

Commits on Jan 12, 2012

  1. compiler: rename unique_{free,dump} to symtab_* 

    Leaving "unique" and "unique_n" for now because I can't decide on a
good name.
    @wpwrak
    wpwrak committed Jan 12, 2012
    Copy the full SHA
    5cdc43b View commit details
    Browse the repository at this point in the history

  2. compiler: optionally warn if using a variable in the wrong section (WIP) 

    This warning is currently only accessible via ptest, with the option
-Wsection.

To do:
- generalize the notification mechanism (currently printf)
- consider also handling variables like x, y, rad
    @wpwrak
    wpwrak committed Jan 12, 2012
    Copy the full SHA
    5fa3a61 View commit details
    Browse the repository at this point in the history

  3. compiler: added per-symbol flags to symbol table

    @wpwrak
    wpwrak committed Jan 12, 2012
    Copy the full SHA
    edecc6e View commit details
    Browse the repository at this point in the history

  4. compiler: -Wundefined to warn about undefined variables (WIP) 

    Limitations:
- complains about iterative calculations like  avg = A*avg+(1-A)*VALUE
- no way to explicitly initialize variables to zero (making the warning
  go away)
    @wpwrak
    wpwrak committed Jan 12, 2012
    Copy the full SHA
    cb6d92a View commit details
    Browse the repository at this point in the history

  5. regression tests: attempt only valid assignments in initial/global se… 

    …ction

Many tests consisted of a single line like  x = something
which are not valid, because "x" is not a per-frame variable.

The next commit will slightly tighten the checks the compiler
performs even without code generation, so we need to make the
test cases a little more correct.
    @wpwrak
    wpwrak committed Jan 12, 2012
    Copy the full SHA
    eaffaac View commit details
    Browse the repository at this point in the history

  6. compiler: allow explicit zero-initialization to suppress warning 

    This is applicable to all variables, not only per-frame variables.
    @wpwrak
    wpwrak committed Jan 12, 2012
    Copy the full SHA
    6e7d6c1 View commit details
    Browse the repository at this point in the history

  7. symtab: rename "syms" to user_syms, for export; minor cleanup

    @wpwrak
    wpwrak committed Jan 12, 2012
    Copy the full SHA
    ab35e12 View commit details
    Browse the repository at this point in the history

  8. compiler: symbol table can now be iterated 

    symtab.h defines a macro forall_syms() that iterates over all symbols.
This means that it's now trivial to implement symtab_dump, and we can
therefore remove it.
    @wpwrak
    wpwrak committed Jan 12, 2012
    Copy the full SHA
    068cfc9 View commit details
    Browse the repository at this point in the history

  9. compiler: -Wundefined now also works for old-style patches 

    Old style assumes everything is implicitly initialized to zero.
We can still report identifiers that are never assigned to, though.

New style allows explicit initialization and thus can report
uninitialized variables immediately when they're read.
    @wpwrak
    wpwrak committed Jan 12, 2012
    Copy the full SHA
    1f53fb2 View commit details
    Browse the repository at this point in the history

  10. parser.y: tok2op wasn't "static", oops

    @wpwrak
    wpwrak committed Jan 12, 2012
    Copy the full SHA
    6259a03 View commit details
    Browse the repository at this point in the history