diff options
| author | Corey Farwell <coreyf@rwell.org> | 2017-03-22 23:38:03 -0400 |
|---|---|---|
| committer | GitHub <noreply@github.com> | 2017-03-22 23:38:03 -0400 |
| commit | 93074187b6c7aeeec6a8fd28574bfad7a4608e1f (patch) | |
| tree | 7447808378b144534bdac51dce3a48670cda23a9 /src/rustllvm/PassWrapper.cpp | |
| parent | 39e4a272049f00f55208c04fa8c6a8dfcdbe4c8a (diff) | |
| parent | 873248d87d1b376c0411d474c665cc6eeefbb16c (diff) | |
| download | rust-93074187b6c7aeeec6a8fd28574bfad7a4608e1f.tar.gz rust-93074187b6c7aeeec6a8fd28574bfad7a4608e1f.zip | |
Rollup merge of #40696 - cramertj:remove-unused-adt-def-code, r=petrochenkov
Remove unused adt-def insertion by constructor DefIndex It looks to me like ADT definitions weren't being looked up by constructor id, and a test run supports my theory. In any case, I'm not sure it would have worked in its current configuration. If I understand correctly, the `adt_def` map entry from constructor id -> adt def would only be present after a successful call to `queries::adt_def::get` with the proper ADT `DefIndex`. Trying to look up an adt_def by the constructor index prior to a successful lookup by ADT index would fail since `item.kind` would be `EntryKind::Fn` (for the constructor function) and so would trigger the `bug!`. r? @nikomatsakis
Diffstat (limited to 'src/rustllvm/PassWrapper.cpp')
0 files changed, 0 insertions, 0 deletions