about summary refs log tree commit diff
path: root/src/rustllvm/PassWrapper.cpp
diff options
context:
space:
mode:
authorbors <bors@rust-lang.org>2013-08-13 16:53:14 -0700
committerbors <bors@rust-lang.org>2013-08-13 16:53:14 -0700
commit433fbe8fcf13724445cc57c85cc454352c969429 (patch)
treeaca4edd4be02a72d29094b699f00fc71dc96552d /src/rustllvm/PassWrapper.cpp
parent9f379329db48e613f910167a8840fd2c13581dd9 (diff)
parent983cc777c5090df18188dd417cdfda8acfc2cb1b (diff)
downloadrust-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