[PATCH] D26855: New unsafe-fp-math implementation for X86 target

[Andrew V. Tischenko via Phabricator via llvm-commits]

avt77 updated this revision to Diff 84535.
avt77 added a comment.

I fixed everything except one comment (see below).  And I collected new perf numbers.  Now I used the following command for bootstrap building:

time make -j 1

As result the reproducing is very well from my point of view. In addition I tried to get numbers accordingly to description in  http://lists.llvm.org/pipermail/llvm-commits/Week-of-Mon-20161017/398831.html . The reproducing is almost the same but the required time is even longer (about 2 hours for every test). Because of that I kept my numbers. The test itself is very simple: I created 2 versions of compiler: the first one was built directly from trunk and the second one was built after applying of my patch. Then with help of every compiler I created 4 bootstraps. The results are below:

  The trunk version of the compiler builds the bootstrap like here (there were 4 starts with "time make -j 1"):
  
  real  91m7.998s
  real  90m23.861s
  real  90m26.154s
  real  90m31.533s
  
  The version of the compiler with my patch builds the bootstrap like here (there were the same 4 starts with "time make -j 1"):
  
  real  90m43.970s
  real 90m7.257s
  real  90m6.671s
  real  90m11.733s

Obviously, the compilation time does not depend on my patch.


https://reviews.llvm.org/D26855

Files:
  include/llvm/CodeGen/MachineCombinerPattern.h
  lib/CodeGen/MachineCombiner.cpp
  lib/CodeGen/SelectionDAG/DAGCombiner.cpp
  lib/Target/X86/X86InstrInfo.cpp
  lib/Target/X86/X86InstrInfo.h
  test/CodeGen/X86/recip-fastmath.ll
  test/CodeGen/X86/recip-fastmath2.ll

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D26855.84535.patch
Type: text/x-patch
Size: 37595 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20170116/43496dd6/attachment.bin>