diff options
| author | bors <bors@rust-lang.org> | 2013-08-13 16:53:14 -0700 |
|---|---|---|
| committer | bors <bors@rust-lang.org> | 2013-08-13 16:53:14 -0700 |
| commit | 433fbe8fcf13724445cc57c85cc454352c969429 (patch) | |
| tree | aca4edd4be02a72d29094b699f00fc71dc96552d /src/rustllvm/PassWrapper.cpp | |
| parent | 9f379329db48e613f910167a8840fd2c13581dd9 (diff) | |
| parent | 983cc777c5090df18188dd417cdfda8acfc2cb1b (diff) | |
| download | rust-433fbe8fcf13724445cc57c85cc454352c969429.tar.gz rust-433fbe8fcf13724445cc57c85cc454352c969429.zip | |
auto merge of #8329 : michaelwoerister/rust/lexical_scopes_alt, r=graydon
This pull request re-implements handling of visibility scopes and source code positions in debug info. It should now be very stable and properly handle + variable shadowing + expanded code (macros and the new for-loop de-sugaring, for example) + variables in the middle of nested scopes + bindings declared in the head of match statement arms. all of which did not work at all or did not work reliably before. Those interested in a more detailed description of the problems at hand, I kindly refer to http://michaelwoerister.github.io/2013/08/03/visibility-scopes.html Why doesn't the `populate_scope_map()` function use `syntax::visit`? Because it would not improve this particular AST walker (see: https://github.com/michaelwoerister/rust/commit/69dc7908490b8b492e9730cb1c34fa83c2ef30eb#commitcomment-3781426) Cheers, Michael
Diffstat (limited to 'src/rustllvm/PassWrapper.cpp')
0 files changed, 0 insertions, 0 deletions
