https://github.com/arsenm commented:

This new codegen looks a lot worse. I think you should just fix the bugs in the 
existing implementation.

The canonicalize / flush handling is plainly broken. I think you should follow 
along with these patches:

Drop the second canonicalize:
https://github.com/ROCm/llvm-project/commit/9a7bc191b9d6430985bce4dce55c3acef5c9fd3a

Defer the flush of denormal:
https://github.com/ROCm/llvm-project/commit/e9198f7d4619eb222f66c1f1d496eee3acdd8511





https://github.com/llvm/llvm-project/pull/162643
_______________________________________________
cfe-commits mailing list
[email protected]
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to