Closed Lugatod closed 7 years ago
@stephenchouca can you take a look at the commit cbrt "hack" ? There is an issue about scalars as parameters of intrinsic functions.
@Lugatod In this context, it would probably make more sense to declare cbrt
as an intrinsic function (Func::Intrinsic) as opposed to an external function (Func::External) in src/intrinsics.cpp, following the example of the tan
intrinsic. The backend would also have to be appropriately updated for this to work (see src/backend/llvm/llvm_backend.cpp:936).
Eventually we should probably change the way we handle scalar parameters and scalars in general.
First validated simit version of Lulesh. Need to work on computation time.