about summary refs log tree commit diff
path: root/src/rustllvm/ExecutionEngineWrapper.cpp
diff options
context:
space:
mode:
authorManish Goregaokar <manishsmail@gmail.com>2015-05-18 08:55:38 +0530
committerManish Goregaokar <manishsmail@gmail.com>2015-05-18 08:55:38 +0530
commit5ed02b5c0eced7c9e4aa72b2096e537bede31df8 (patch)
treec8c050b2e23bd046792caaf22df68b981fe7526c /src/rustllvm/ExecutionEngineWrapper.cpp
parent252027768eac85cfd98427ba2cbf1d51bfc77d53 (diff)
parente70c8584d562c86fdca9c98895fb981683d18859 (diff)
downloadrust-5ed02b5c0eced7c9e4aa72b2096e537bede31df8.tar.gz
rust-5ed02b5c0eced7c9e4aa72b2096e537bede31df8.zip
Rollup merge of #25537 - carols10cents:grammar-oops, r=alexcrichton
Hiiii soooo I'm trying to get the reference grammar and associated tests running again, and I swear I tested before but I must have had multiple things going on when I did, because the change I made in #25137 to verify.rs is totally wrong. The RustLexer.tokens file that antlr generates has two sections:

```
EQ=1
LT=2
LE=3
EQEQ=4
NE=5
...
COMMENT=56
SHEBANG=57
UTF8_BOM=58
'='=1
'<'=2
'<='=3
'=='=4
...
```

and verify.rs is only interested in the first half-- the `continue` is to ignore the second half. In 9c7d5ae, I made it panic instead. I was trying to make sure verify.rs handled everything that might happen in the first half and complain if it didn't. That would mean the reference grammar was out of sync with at least verify.rs, if not the real grammar. But it's totally ok for verify.rs to not handle the entire second half of the file.

I'm sorry for breaking this :( Good thing these tests aren't being run regularly yet...? :flushed: 
Diffstat (limited to 'src/rustllvm/ExecutionEngineWrapper.cpp')
0 files changed, 0 insertions, 0 deletions