search

EnzymeAD / rust

A rust fork to work towards Enzyme integration
https://www.rust-lang.org
Other
72 stars 8 forks source link

more precise memcpy tt #140

Closed ZuseZ4 closed 4 months ago

ZuseZ4 commented 4 months ago

Third arg is the size, and thus always an int.

wsmoses commented 4 months ago

This should already be assumed why is this needed

ZuseZ4 commented 4 months ago

I was also surprised since I would have expected Enzyme core to handle this. I had enzyme complaining about can not deduce type of memcpy in a specific case where I had given tt for the first two args (src and dst, both -1:ptr and -1:0 to -1:72 being float). So I added this and now it went away. Let me see if I can reproduce it by undoing this pr.