Intrinsics are not going to work any time soon

0xProject · Mar 18, 2020 · 8b465c5 · 8b465c5
1 parent b4caddc
commit 8b465c5
Show file tree

Hide file tree

Showing 3 changed files with 4 additions and 87 deletions.
diff --git a/algebra/u256/src/algorithms/assembly.rs b/algebra/u256/src/algorithms/assembly.rs
@@ -2,6 +2,10 @@
 use crate::{MontgomeryParameters, U256};
 use std::mem::MaybeUninit;
 
+// TODO: Intrinsics based approach usind AD_X is currently blocked on LLVM.
+// See <https://github.com/rust-lang/stdarch/issues/666>
+// See <https://bugs.llvm.org/show_bug.cgi?id=41546>
+
 // For instruction timings and through puts
 // See <https://gmplib.org/~tege/x86-timing.pdf>
 

diff --git a/algebra/u256/src/algorithms/intrinsics.rs b/algebra/u256/src/algorithms/intrinsics.rs
diff --git a/algebra/u256/src/algorithms/mod.rs b/algebra/u256/src/algorithms/mod.rs
@@ -1,6 +1,5 @@
 pub(crate) mod assembly;
 mod binary_operator_macros;
-pub(crate) mod intrinsics;
 mod knuth_division;
 mod lehmer_gcd;
 mod limb_operations;