Enzyme icon indicating copy to clipboard operation
Enzyme copied to clipboard

Ubuntu22

Open tgymnich opened this issue 3 years ago • 5 comments

tgymnich avatar Jun 09 '22 14:06 tgymnich

Would it make sense to set up a kind of support policy in order to limit what we test (and thus especially CI time)? 18.04 is more than 4 yrs old and EOL in less than a year, so I wonder if we can't just replace it by 22.04?

ZuseZ4 avatar Jun 09 '22 16:06 ZuseZ4

Supporting ubuntu 18.04 until its EOL and replacing it with 22.04 sound like a good plan.

tgymnich avatar Jun 10 '22 14:06 tgymnich

Failures in the integration tests for LLVM 11,12 and 13 in Integration/ReverseMode/eigensumsqdyn-notmp.cpp.

warning: <unknown>:0:0: failed to deduce type of copy   call void @llvm.memcpy.p0i8.p0i8.i64(i8* nonnull align 8 dereferenceable(32) %5, i8* nonnull align 8 dereferenceable(32) %6, i64 32, i1 false) #18
Failed Tests (8):
[943](https://github.com/EnzymeAD/Enzyme/actions/runs/3182278630/jobs/5188028238#step:7:944)
  Enzyme :: Integration/ReverseMode/eigensumsqdyn-notmp.cpp
[944](https://github.com/EnzymeAD/Enzyme/actions/runs/3182278630/jobs/5188028238#step:7:945)
  Enzyme :: Integration/ReverseMode/eigensumsqdyn.cpp
[945](https://github.com/EnzymeAD/Enzyme/actions/runs/3182278630/jobs/5188028238#step:7:946)
  Enzyme :: Integration/ReverseMode/eigentensor.cpp
[946](https://github.com/EnzymeAD/Enzyme/actions/runs/3182278630/jobs/5188028238#step:7:947)
  Enzyme :: Integration/ReverseMode/eigentensorfull.cpp
[947](https://github.com/EnzymeAD/Enzyme/actions/runs/3182278630/jobs/5188028238#step:7:948)
  Enzyme :: Integration/ReverseMode/integrateexp.cpp
[948](https://github.com/EnzymeAD/Enzyme/actions/runs/3182278630/jobs/5188028238#step:7:949)
  Enzyme :: Integration/ReverseMode/simpleeigenstatic-made.cpp
[949](https://github.com/EnzymeAD/Enzyme/actions/runs/3182278630/jobs/5188028238#step:7:950)
  Enzyme :: Integration/ReverseMode/simpleeigenstatic-vec.cpp
[950](https://github.com/EnzymeAD/Enzyme/actions/runs/3182278630/jobs/5188028238#step:7:951)
  Enzyme :: Integration/ReverseMode/simpleeigenstatic.cpp

tgymnich avatar Jun 11 '22 14:06 tgymnich

We should investigate the Type analysis failures to see if we can make them detectable/fixable.

Otherwise lgtm

wsmoses avatar Sep 04 '22 03:09 wsmoses

The segfault from Enzyme :: Integration/ReverseMode/integrateexp.cpp needs to be fixed before merging. Anyone have time to investigate?

wsmoses avatar Oct 04 '22 22:10 wsmoses