Skip to content

Commit b2c9fc2

Browse files
authored
Unrolled build for rust-lang#135027
Rollup merge of rust-lang#135027 - Flakebi:remove-range-workaround, r=compiler-errors Remove range-metadata amdgpu workaround Range metadata was disabled for amdgpu due to a backend bug. I did not encounter any problems when removing the workaround to enable range metadata (tried compiling `core` and `alloc`), so I assume this has been fixed in LLVM in the last years. Remove the workaround to re-enable range metadata. Tracking issue: rust-lang#135024
2 parents 1b2745d + 56bf673 commit b2c9fc2

File tree

1 file changed

+0
-8
lines changed

1 file changed

+0
-8
lines changed

compiler/rustc_codegen_llvm/src/builder.rs

-8
Original file line numberDiff line numberDiff line change
@@ -610,14 +610,6 @@ impl<'a, 'll, 'tcx> BuilderMethods<'a, 'tcx> for Builder<'a, 'll, 'tcx> {
610610
}
611611

612612
fn range_metadata(&mut self, load: &'ll Value, range: WrappingRange) {
613-
if self.sess().target.arch == "amdgpu" {
614-
// amdgpu/LLVM does something weird and thinks an i64 value is
615-
// split into a v2i32, halving the bitwidth LLVM expects,
616-
// tripping an assertion. So, for now, just disable this
617-
// optimization.
618-
return;
619-
}
620-
621613
if self.cx.sess().opts.optimize == OptLevel::No {
622614
// Don't emit metadata we're not going to use
623615
return;

0 commit comments

Comments
 (0)