search

Quuxplusone / LLVMBugzillaTest

0 stars 0 forks source link

Merge r319025 into the 5.0 branch : [AMDGPU] Add custom lowering for llvm.log{,10}.{f16,f32} intrinsics #34415

Open Quuxplusone opened 6 years ago

Quuxplusone commented 6 years ago
Bugzilla Link PR35442
Status NEW
Importance P enhancement
Reported by Vedran Miletic (vedran@miletic.net)
Reported on 2017-11-27 15:53:30 -0800
Last modified on 2017-11-28 18:50:36 -0800
Version 5.0
Hardware All All
CC jv356@scarletmail.rutgers.edu, llvm-bugs@lists.llvm.org, tstellar@redhat.com
Fixed by commit(s) rL319025
Attachments
Blocks PR34492
Blocked by
See also

Is it OK to merge the following revision(s) to the 5.0 branch?

Quuxplusone commented 6 years ago

https://reviews.llvm.org/rL319025

Quuxplusone commented 6 years ago

Some test cases fail when I apply this to the release_50 branch would you be able to supply a patch that applies cleanly and doesn't regress tests?

Also, it's pretty late in the process, so I'm not going to hold -rc2 back waiting for this to be resolved.

Quuxplusone commented 6 years ago

This was submitted too late to make it into 5.0.1-rc2, so it won't be in the release unless it is determined to be a critical fix.