Skip to content

Commit

Permalink
Revert "cg_llvm: fewer_names in uncached_llvm_type"
Browse files Browse the repository at this point in the history
This reverts commit fa01ce8.
  • Loading branch information
davidtwco committed Dec 17, 2020
1 parent caeb333 commit 88dc58f
Show file tree
Hide file tree
Showing 2 changed files with 3 additions and 10 deletions.
10 changes: 1 addition & 9 deletions compiler/rustc_codegen_llvm/src/type_of.rs
Expand Up @@ -40,9 +40,7 @@ fn uncached_llvm_type<'a, 'tcx>(
// FIXME(eddyb) producing readable type names for trait objects can result
// in problematically distinct types due to HRTB and subtyping (see #47638).
// ty::Dynamic(..) |
ty::Adt(..) | ty::Closure(..) | ty::Foreign(..) | ty::Generator(..) | ty::Str
if !cx.sess().fewer_names() =>
{
ty::Adt(..) | ty::Closure(..) | ty::Foreign(..) | ty::Generator(..) | ty::Str => {
let mut name = with_no_trimmed_paths(|| layout.ty.to_string());
if let (&ty::Adt(def, _), &Variants::Single { index }) =
(layout.ty.kind(), &layout.variants)
Expand All @@ -58,12 +56,6 @@ fn uncached_llvm_type<'a, 'tcx>(
}
Some(name)
}
ty::Adt(..) => {
// If `Some` is returned then a named struct is created in LLVM. Name collisions are
// avoided by LLVM (with increasing suffixes). If rustc doesn't generate names then that
// can improve perf.
Some(String::new())
}
_ => None,
};

Expand Down
3 changes: 2 additions & 1 deletion src/test/ui/issues/issue-75763.rs
@@ -1,4 +1,5 @@
// build-pass
// ignore-test
// FIXME(const_generics): This test causes an ICE after reverting #76030.

#![allow(incomplete_features)]
#![feature(const_generics)]
Expand Down

0 comments on commit 88dc58f

Please sign in to comment.