New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Function ABI is irrelevant for reachability #123063
Conversation
rustbot has assigned @michaelwoerister. Use |
This comment has been minimized.
This comment has been minimized.
9d5c91d
to
955f762
Compare
The code was originally introduced in 4ddeef3. It looks like it was made ineffective since we have distinction between Rust and C symbol export level (it exports symbols at Rust level, but it is used only for crate type with C level threshold). |
I agree, it looks like that can never make a difference, i.e. the code does something akin to: let exported = symbols.filter(|s| s.abi != Rust || extern(s)).filter(|s| extern(s)) @bors r+ |
…iaskrgr Rollup of 6 pull requests Successful merges: - rust-lang#123063 (Function ABI is irrelevant for reachability) - rust-lang#123096 (Don't check match scrutinee of postfix match for unused parens) - rust-lang#123146 (Use compiletest directives instead of manually checking TARGET / tools) - rust-lang#123160 (remove `def_id_to_node_id` in ast lowering) - rust-lang#123162 (Correctly get complete intra-doc link data) - rust-lang#123164 (Bump Unicode printables to version 15.1, align to unicode_data) r? `@ghost` `@rustbot` modify labels: rollup
Rollup merge of rust-lang#123063 - tmiasko:reachability-abi, r=michaelwoerister Function ABI is irrelevant for reachability
No description provided.