Skip to content

Commit 1e17e6b

Browse files
committed
Use globals instead of metadata, since metadata isn't emitted in debug builds
1 parent 981353c commit 1e17e6b

File tree

1 file changed

+24
-24
lines changed

1 file changed

+24
-24
lines changed

compiler/rustc_codegen_llvm/src/builder/autodiff.rs

Lines changed: 24 additions & 24 deletions
Original file line numberDiff line numberDiff line change
@@ -13,7 +13,7 @@ use crate::builder::{Builder, PlaceRef, UNNAMED};
1313
use crate::context::SimpleCx;
1414
use crate::declare::declare_simple_fn;
1515
use crate::llvm;
16-
use crate::llvm::{Metadata, TRUE, Type};
16+
use crate::llvm::{TRUE, Type};
1717
use crate::value::Value;
1818

1919
pub(crate) fn adjust_activity_to_abi<'tcx>(
@@ -159,32 +159,32 @@ fn match_args_from_caller_to_enzyme<'ll, 'tcx>(
159159
let mut outer_pos: usize = 0;
160160
let mut activity_pos = 0;
161161

162-
let enzyme_const = cx.create_metadata(b"enzyme_const");
163-
let enzyme_out = cx.create_metadata(b"enzyme_out");
164-
let enzyme_dup = cx.create_metadata(b"enzyme_dup");
165-
let enzyme_dupv = cx.create_metadata(b"enzyme_dupv");
166-
let enzyme_dupnoneed = cx.create_metadata(b"enzyme_dupnoneed");
167-
let enzyme_dupnoneedv = cx.create_metadata(b"enzyme_dupnoneedv");
162+
let global_const = cx.declare_global("enzyme_const", cx.type_ptr());
163+
let global_out = cx.declare_global("enzyme_out", cx.type_ptr());
164+
let global_dup = cx.declare_global("enzyme_dup", cx.type_ptr());
165+
let global_dupv = cx.declare_global("enzyme_dupv", cx.type_ptr());
166+
let global_dupnoneed = cx.declare_global("enzyme_dupnoneed", cx.type_ptr());
167+
let global_dupnoneedv = cx.declare_global("enzyme_dupnoneedv", cx.type_ptr());
168168

169169
while activity_pos < inputs.len() {
170170
let diff_activity = inputs[activity_pos as usize];
171171
// Duplicated arguments received a shadow argument, into which enzyme will write the
172172
// gradient.
173-
let (activity, duplicated): (&Metadata, bool) = match diff_activity {
173+
let (activity, duplicated): (&llvm::Value, bool) = match diff_activity {
174174
DiffActivity::None => panic!("not a valid input activity"),
175-
DiffActivity::Const => (enzyme_const, false),
176-
DiffActivity::Active => (enzyme_out, false),
177-
DiffActivity::ActiveOnly => (enzyme_out, false),
178-
DiffActivity::Dual => (enzyme_dup, true),
179-
DiffActivity::Dualv => (enzyme_dupv, true),
180-
DiffActivity::DualOnly => (enzyme_dupnoneed, true),
181-
DiffActivity::DualvOnly => (enzyme_dupnoneedv, true),
182-
DiffActivity::Duplicated => (enzyme_dup, true),
183-
DiffActivity::DuplicatedOnly => (enzyme_dupnoneed, true),
184-
DiffActivity::FakeActivitySize(_) => (enzyme_const, false),
175+
DiffActivity::Const => (global_const, false),
176+
DiffActivity::Active => (global_out, false),
177+
DiffActivity::ActiveOnly => (global_out, false),
178+
DiffActivity::Dual => (global_dup, true),
179+
DiffActivity::Dualv => (global_dupv, true),
180+
DiffActivity::DualOnly => (global_dupnoneed, true),
181+
DiffActivity::DualvOnly => (global_dupnoneedv, true),
182+
DiffActivity::Duplicated => (global_dup, true),
183+
DiffActivity::DuplicatedOnly => (global_dupnoneed, true),
184+
DiffActivity::FakeActivitySize(_) => (global_const, false),
185185
};
186186
let outer_arg = outer_args[outer_pos];
187-
args.push(cx.get_metadata_value(activity));
187+
args.push(activity);
188188
if matches!(diff_activity, DiffActivity::Dualv) {
189189
let next_outer_arg = outer_args[outer_pos + 1];
190190
let elem_bytes_size: u64 = match inputs[activity_pos + 1] {
@@ -244,7 +244,7 @@ fn match_args_from_caller_to_enzyme<'ll, 'tcx>(
244244
assert_eq!(cx.type_kind(next_outer_ty3), TypeKind::Integer);
245245
args.push(next_outer_arg2);
246246
}
247-
args.push(cx.get_metadata_value(enzyme_const));
247+
args.push(global_const);
248248
args.push(next_outer_arg);
249249
outer_pos += 2 + 2 * iterations;
250250
activity_pos += 2;
@@ -353,13 +353,13 @@ pub(crate) fn generate_enzyme_call<'ll, 'tcx>(
353353
let mut args = Vec::with_capacity(num_args as usize + 1);
354354
args.push(fn_to_diff);
355355

356-
let enzyme_primal_ret = cx.create_metadata(b"enzyme_primal_return");
356+
let global_primal_ret = cx.declare_global("enzyme_primal_return", cx.type_ptr());
357357
if matches!(attrs.ret_activity, DiffActivity::Dual | DiffActivity::Active) {
358-
args.push(cx.get_metadata_value(enzyme_primal_ret));
358+
args.push(global_primal_ret);
359359
}
360360
if attrs.width > 1 {
361-
let enzyme_width = cx.create_metadata(b"enzyme_width");
362-
args.push(cx.get_metadata_value(enzyme_width));
361+
let global_width = cx.declare_global("enzyme_width", cx.type_ptr());
362+
args.push(global_width);
363363
args.push(cx.get_const_int(cx.type_i64(), attrs.width as u64));
364364
}
365365

0 commit comments

Comments
 (0)