Skip to content

Commit

Permalink
Comment change suggestions from oli-obk's code review
Browse files Browse the repository at this point in the history
Co-authored-by: Oli Scherer <[email protected]>
  • Loading branch information
NCGThompson and oli-obk committed Jan 16, 2024
1 parent ba5dcc7 commit 61faffb
Show file tree
Hide file tree
Showing 2 changed files with 1 addition and 3 deletions.
2 changes: 1 addition & 1 deletion compiler/rustc_codegen_cranelift/src/intrinsics/mod.rs
Original file line number Diff line number Diff line change
Expand Up @@ -442,7 +442,7 @@ fn codegen_regular_intrinsic_call<'tcx>(
sym::is_val_statically_known => {
intrinsic_args!(fx, args => (_a); intrinsic);

// FIXME impliment intrinsic or guarantee propogation of false
// FIXME actually return `true` in case the argument is a constant
let res = fx.bcx.ins().iconst(cranelift_codegen::ir::types::I8, false as i64);
ret.write_cvalue(fx, CValue::by_val(res, ret.layout()));
}
Expand Down
2 changes: 0 additions & 2 deletions tests/codegen/is_val_statically_known.rs
Original file line number Diff line number Diff line change
@@ -1,5 +1,3 @@
// #[cfg(bootstrap)]
// ignore-stage1
// compile-flags: --crate-type=lib -Zmerge-functions=disabled

#![feature(core_intrinsics)]
Expand Down

0 comments on commit 61faffb

Please sign in to comment.