<div dir="ltr"><div class="gmail_extra"><br><br><div class="gmail_quote">On Wed, May 7, 2014 at 10:30 AM, Bob Wilson <span dir="ltr"><<a href="mailto:bob.wilson@apple.com" target="_blank">bob.wilson@apple.com</a>></span> wrote:<br>

<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div style="word-wrap:break-word"><div><br><div><div>
On May 7, 2014, at 10:02 AM, Alexey Samsonov <<a href="mailto:samsonov@google.com" target="_blank">samsonov@google.com</a>> wrote:</div><br><blockquote type="cite"><div dir="ltr"><div class="gmail_extra"><br><div class="gmail_quote">

On Wed, May 7, 2014 at 8:28 AM, Hal Finkel <span dir="ltr"><<a href="mailto:hfinkel@anl.gov" target="_blank">hfinkel@anl.gov</a>></span> wrote:<br>
<blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex"><div>----- Original Message -----<br>
> From: "Anton Korobeynikov" <<a href="mailto:anton@korobeynikov.info" target="_blank">anton@korobeynikov.info</a>><br>
> To: "Hal Finkel" <<a href="mailto:hfinkel@anl.gov" target="_blank">hfinkel@anl.gov</a>><br>
</div><div>> Cc: "Alexey Samsonov" <<a href="mailto:samsonov@google.com" target="_blank">samsonov@google.com</a>>, "LLVM Developers Mailing List" <<a href="mailto:llvmdev@cs.uiuc.edu" target="_blank">llvmdev@cs.uiuc.edu</a>><br>



> Sent: Thursday, May 1, 2014 1:56:04 AM<br>
> Subject: Re: [LLVMdev] Is it ok to nuke fpcmp and llvm-PerfectShuffle utils?<br>
><br>
</div><div>> >> Ouch, indeed... Is it ok it has no build rules?<br>
> ><br>
> > I think that it should have build rules. If nothing else, can you<br>
> > file a bug report on this?<br>
> It is building using configure + make. In good old days the perfect<br>
> shuffle tables for PPC were generated through the normal build<br>
> process<br>
> like .td files. However, this was pretty useless - they do not change<br>
> and thus perfectshuffle was excluded from the build process.<br>
<br>
</div>Agreed, but even if it is not built by default, we should have build rules for it; and given that more users and developers are moving to cmake-based builds, we should have build rules in cmake too.<br></blockquote>


<div><br></div><div>I've added it to CMake build in r208225.</div><div><br></div><div>What about fpcmp binary? Does anyone still use it?</div></div></div></div></blockquote><br></div></div><div>It is still used in the test-suite.</div>

<div><br></div><div>From Makefile.rules:</div><div><br></div><div>FPCMP     = $(PROJ_OBJ_ROOT)/tools/fpcmp</div></div></blockquote><div><br></div><div><br></div><blockquote class="gmail_quote" style="margin:0px 0px 0px 0.8ex;border-left-width:1px;border-left-color:rgb(204,204,204);border-left-style:solid;padding-left:1ex">

<div style="word-wrap:break-word"><div><br></div><div>and from Makefile.programs:</div><div><br></div><div># DIFFPROG - The program used to diff the output<br>DIFFPROG := $(PROGDIR)/DiffOutput.sh "$(FPCMP) $(TOLERANCEOPT)"<br>

</div></div></blockquote><div><br></div><div>LLVM's fpcmp I am talking about is located under utils/fpcmp.</div><div><br></div><div>Looks like test-suite has its own version of it in projects/test-suite/tools/fpcmp.c.</div>
</div><br clear="all"><div>
<br></div>-- <br><div dir="ltr"><div>Alexey Samsonov, Mountain View, CA</div></div>
</div></div>