<html><head><meta http-equiv="Content-Type" content="text/html charset=us-ascii"></head><body style="word-wrap: break-word; -webkit-nbsp-mode: space; -webkit-line-break: after-white-space;"><br><div><div>On May 7, 2014, at 10:02 AM, Alexey Samsonov <<a href="mailto:samsonov@google.com">samsonov@google.com</a>> wrote:</div><br class="Apple-interchange-newline"><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:0 0 0 .8ex;border-left:1px #ccc solid;padding-left:1ex"><div class="">----- Original Message -----<br>
> From: "Anton Korobeynikov" <<a href="mailto:anton@korobeynikov.info">anton@korobeynikov.info</a>><br>
> To: "Hal Finkel" <<a href="mailto:hfinkel@anl.gov">hfinkel@anl.gov</a>><br>
</div><div class="">> Cc: "Alexey Samsonov" <<a href="mailto:samsonov@google.com">samsonov@google.com</a>>, "LLVM Developers Mailing List" <<a href="mailto:llvmdev@cs.uiuc.edu">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 class="">> >> 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>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><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></body></html>