-
Notifications
You must be signed in to change notification settings - Fork 788
[ESIMD] Break @llvm.fmuladd back into mul/add to satisfy VC BE. #5075
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Conversation
Signed-off-by: Konstantin S Bobrovsky <[email protected]>
With the latest pulldown default FE handling of FP changed and it started to generate fmuladd here. |
Please check what is wrong with ESIMD/regression/dgetrf.cpp - it failed in CI, couldn't that be caused by this patch? |
ESIMD failures in Jenkins are "unexpected pass"-es, which are expected - this patch fixes the regressions on them. [2021-12-03T06:26:18.803Z] Failed Tests (5): [2021-12-03T06:09:03.533Z] Unexpectedly Passed Tests (3): |
It is also unexpected XPASS, the reason as the same as for mandelbrot - it started to generate fmuladd, was marked as XFAIL, now this patch fixes it. |
Ok, understood. BTW, then it makes sense fixing the LIT tests (remove XFAIL) and run the command/comment int this 5075 such line: "/verify with REFERENCE_TO_LIT_FIX_PR" |
After re-running
good point, will do |
Rerunning the precommit left only the 3 "unexpected pass" failures above. |
Accompanying test fix: intel/llvm-test-suite#600 |
"/verify with #5075" in intel/llvm-test-suite#600 did not work - it did not make CI pick this my patch for testing. Nevertheless, "Unexepected pass" on the 3 tests w/o regression is 100% evidence that the patch does the right thing, so I'm merging. |
…suite#600) Signed-off-by: Konstantin S Bobrovsky <[email protected]>
VC BE does not support many of std llvm intrinsics for now, so need a w/a.
This patch handles
fmuladd
.This fixes regression on mandelbrot.cpp E2E test.
Signed-off-by: Konstantin S Bobrovsky [email protected]