search for: rsqrt_modified_167927

...39;re adding isn't in the right place; it should probably be a target-specific DAGCombine, somewhere in X86ISelLowering.cpp. Also, please take the time to read http://llvm.org/docs/CodingStandards.html . -Eli -------------- next part -------------- A non-text attachment was scrubbed... Name: rsqrt_modified_167927.patch Type: application/octet-stream Size: 12306 bytes Desc: rsqrt_modified_167927.patch URL: <http://lists.llvm.org/pipermail/llvm-dev/attachments/20121203/69610942/attachment.obj> -------------- next part -------------- An embedded and charset-unspecified text was scrubbed... Name: rsqrt-de...

On Wed, Nov 14, 2012 at 10:43 PM, Chakraborty, Soham <Soham.Chakraborty at amd.com> wrote: > Hi, > > > > We have implemented the rsqrt instruction generation for X86 target > architecture. We have introduced a flag -fp-rsqrt flag which controls the > generatation of X86 rsqrt instruction generation. > > We have observed minor effects on precision due to rsqrt and

Hi, We have implemented the rsqrt instruction generation for X86 target architecture. We have introduced a flag -fp-rsqrt flag which controls the generatation of X86 rsqrt instruction generation. We have observed minor effects on precision due to rsqrt and hence has put these transformations under the mentioned flag. Note that -fp-rsqrt is only enabled with -enable-unsafe-fp-math flag presently.