[PATCH] D29942: Add custom lowering for llvm.log{, 10}.{f16, f32} intrinsics

Tue Apr 18 10:20:15 PDT 2017

rivanvx updated this revision to Diff 95598.
rivanvx marked 3 inline comments as done.
rivanvx added a comment.

Sorry for the delay, updated patch, now passes tests, addressed all the comments. I would appreciate if you could check if I got all the R_ and A_ prefixes correctly (it's just variable naming, but there is some logic behind it -- register vs address, I presume?) in f16 tests.

https://reviews.llvm.org/D29942

Files:
  lib/Target/AMDGPU/AMDGPUISelLowering.cpp
  lib/Target/AMDGPU/AMDGPUISelLowering.h
  test/CodeGen/AMDGPU/llvm.log.f16.ll
  test/CodeGen/AMDGPU/llvm.log.ll
  test/CodeGen/AMDGPU/llvm.log10.f16.ll
  test/CodeGen/AMDGPU/llvm.log10.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D29942.95598.patch
Type: text/x-patch
Size: 18370 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170418/22d67b8c/attachment-0001.bin>