Fix DIP instruction name typos in guest_amd64_toIR.c.

DIP would print the wrong instruction name for blendps, vroundpd and
vpinsrq. Which would be confusing when trying to debug the disassembly.
These were probably typos or copy/paste errors since they would print
very similar instruction names.

Fix memory leak in launcher-linux.c

When the clientname argument is not a full absolute path we reconstruct
the full path using $PATH. This reconstructed full path would be leaked.
This is small and insignificant. But valgrind should give a good example.
So free the path again when we are done and allocated the memory.
Also don't print the path twice in the debug output if we didn't need to
construct a new different path for it.

vgdb: Fix read check in report_pid.

When read fails it will return -1. In which case we might assign
cmdline[sz] = 0 and print a garbage cmdline. Fix the test to check
the return value is > 0.

Fix memcheck/tests/linux/sys-copy_file_range open call (mode).

sys-copy_file_range.c calls open with O_CREAT flag and so must provide
a mode argument. valgrind memcheck actually caught this ommission on
some arches (fedora rawhide i686 specifically).

This is a small additional fixup for
https://bugs.kde.org/show_bug.cgi?id=407218

Make memcheck/tests/x86-linux/scalar test work under root.

Running the testsuite as root isn't really recommended.
But lets not make tests fail unnecessarily when running as root.
Similar to the arm64-linux/scalar fixes. Plus 32bit variants that
don't exist on arm64.

Pass really invalid arguments to setuid[32], setgid[32], acct, fchown[32].
Make setresgid[32], setresuid[32], setregid[32], setreuid[32] always succeed.

Make memcheck/tests/arm64-linux/scalar test work under root.

Running the testsuite as root isn't really recommended.
But lets not make tests fail unnecessarily when running as root.
Pass really invalid arguments to setuid, setgid, acct and fchown.
Make setresgid, setresuid, setregid and setreuid always succeed.

aarch64 (arm64) isn't a supported architecture for exp-sgcheck.

exp-sgcheck/pc_main.c contains:

   #if defined(VGA_arm) || defined(VGA_arm64)
      VG_(printf)("SGCheck doesn't work on ARM yet, sorry.\n");
      VG_(exit)(1);
   #endif

But exp-sgcheck/tests/is_arch_supported checked against uname -m
which returns aarch64 (not arm64). Fix the test check so the
exp-sgcheck tests are skipped instead of producing failures.

Add a test for the new gdbserver adddress[length] syntax.

Have gdbserver accepts the syntax address[length]

The syntax address[length] can be used in all the gdbserer monitor
commands that need an address and optional length argument.

This commit also fixes an error message, and removes trailing whitespaces
in m_gdbserver.c

Explicitly make testcase variable for sys-copy_file_range undefined.

On some systems an extra warning could occur when a variable in
the memcheck/tests/linux/sys-copy_file_range testcase was undefined,
but (accidentially) pointed to known bad memory. Fix by defining the
variable as 0, but then marking it explicitly undefined using memcheck
VALGRIND_MAKE_MEM_UNDEFINED.

Followup for https://bugs.kde.org/show_bug.cgi?id=407218

mips: hook linux copy_file_range syscall

Allow copy_file_range syscalls on MIPS32 and MIPS64.
Update .gitignore.

Related Bugzilla issue - KDE #407218.

mips64: allow Loongson baseline

Allow Loongson baseline on MIPS64. This fixes KDE #406824.

Intercept stpcpy also in ld.so for arm64

On other arches stpcpy () is intercepted for both libc.so and ld.so.
But not on arm64, where it is only intercepted for libc.so.

This can cause memcheck warnings about the use of stpcpy () in ld.so
when called through dlopen () because ld.so contains its own copy of
that functions.

Fix by introducing VG_Z_LD_LINUX_AARCH64_SO_1 (the encoded name of
ld.so on arm64) and using that in vg_replace_strmem.c to intercept
stpcpy.

https://bugs.kde.org/show_bug.cgi?id=407307

Hook linux copy_file_range syscall on arm.

Add support for the copy_file_range syscall

Support amd64, x86, arm64, ppc64, ppc32 and s390x architectures.
Also add sys-copy_file_range test case.

Make the list of horrible filter_gdb sed expressions somewhat less horrible

The sed expressions and the comments of these expression were at 2 different
places, making this already horrible list of expressions even more
horrible to understand/maintain.

So, restructure to allow the comments for an expression be close
to the expression.

mips: fix file permission of guest_mips_toIR.c

Revert accidentally modified file permissions back to 0644.

mips: fix mips32r6 and mips64r6 compilation issue

Add missing variable declarations.
Modify local_sys_write_stderr to use movn if available, and use
seleqz/selnez instructions otherwise.

filter_gdb: add regexp to filter out names which starts with a "."

such names are used for "function descriptors" on ppc64

https://bugs.kde.org/show_bug.cgi?id=406561

Prepare NEWS for 3.16.0

So that the fixed bugs and changes can be documented directly.

Add missing outer argument when self-hosting valgrind.

The outer valgrind tries to run libc and libcxx free res functions
in the guest (which in case of self-hosting is a valgrind tool).
And that fails.

So, disable running such free res functions.

Fix minor leaks in dhat.

Only leaks in dh_fini, and only once.
But fixing these leaks is easier than maintain suppression entries.

Advance the head to 3.16.0.GIT.

drd/tests/tsan_unittest: Avoid that this test reads from uninitialized memory

Reported-by: Philippe Waroquiers <philippe.waroquiers@skynet.be>

Second attempt at a 3.15.0 final version.

Bug 406465 - arm64 instruction selector fails on "t0 = <expr>" where <expr> has type Ity_F16.

-> 3.15.0 final

none/tests/amd64-linux/map_32bits.vgtest fails too easily

On various systems none/tests/amd64-linux/map_32bits.vgtest fails with:
first mmap: Cannot allocate memory.

The problem is that the --aspace-minaddr is too tight. Newer glibc seem
to mmap some memory and so even our first mmap with MMAP_32BIT will fail.

The solution is to make a bit more memory < 2GB available.
If there is 16MB available the test always seems to succeed without
needing too many tries. The original 256K is too low.

https://bugs.kde.org/show_bug.cgi?id=406422

filter_libc: remove the line holding the futex syscall error entirely

The current filter might leave empty lines behind.
This is caused by the fact that glibc used to not include a newline
in the message. But since glibc 2.29 it does.

https://bugs.kde.org/show_bug.cgi?id=405205
Reported-by: Stefan Maksimovic <stefan.maksimovic@rt-rk.com>

-> 3.15.0.RC2

gdbserver_tests fails because of gdb output change.

The 'done.' in "Reading symbols from <exe>...done." is optional.

https://bugs.kde.org/show_bug.cgi?id=406357

Reported-by: Roger Light <roger@atchoo.org>

gdbserver_tests: adjust filtering for the new gdb output

mcsigpass and mcsignopass tests fails due to the slightly different
gdb output. Filter the tests output to make it the same for different
gdb version. Change mcsigpass and mcsignopass .exp to fit filtered
.out.

More generic regexp by Stefan Maksimovic <stefan.maksimovic@rt-rk.com>.

https://bugs.kde.org/show_bug.cgi?id=406355

dhat is broken on x86 (32bit)

dhat will crash on x86 (32bit) because it doesn't handle Vg_CoreClientReq
in its track_post_mem_write.

dh_handle_noninsn_write will be called with Vg_CoreClientReq during
shutdown when trying to setup an argument for the freeres_wrapper().
On other architectures this will be done through a register, but on
x86 we setup the argument on the stack. See final_tidyup () in
coregrind/m_main.c.

Just handle it like a syscall memory argument write.

https://bugs.kde.org/show_bug.cgi?id=406354

More memcheck/tests/libstdc++.supp suppressions

memcheck/tests/leak_cpp_interior might fail because depending on compiler
and arch used the suppression file used doesn't catch some allocations
through the DT_INIT_ARRAY functions.

That memory doesn't actually leak, it gets freed by the
__gnu_cxx::__freeres hook. But since this testcase shows leaks while
running, not after the program ends, it needs to suppress these allocations.

https://bugs.kde.org/show_bug.cgi?id=406360

Add missing cachegrind/tests/a.c to EXTRA_DIST.

cachegrind/callgrind fails ann[12] tests because of missing a.c

These testcases fail because the dist tar is missing the a.c
(auto-annotated) source file. Fix by adding it to EXTRA_DIST.

https://bugs.kde.org/show_bug.cgi?id=406352

GET_STARTREGS for s390: fix register constraint (per comments from Andreas Arnez).

-> 3.15.0.RC1

Fix broken links in the Massif manual.

Misc doc updates for upcoming 3.15.0 release.

Update __VALGRIND_MINOR__ for the upcoming 3.15 release.

Finalise bug-fix status notes.

Finalise wording w.r.t. the MPL.

Mention --num-callers more in DHAT docs.

Mention massif-visualizer in the Massif docs.

Install dhat viewer files in libexec.

libexec seems a better location than libdir. libexec is for internal
binaries or scripts that are not intended to be executed directly.

If we want to change the location again it is now simple.
Just change the dhat/Makefile.am dhatdir variable.

Add arm64 do_syscall_WRK implementation to memcheck/tests/leak-segv-jmp.

On arm64-linux memcheck/tests/leak-segv-jmp would fail because there
was no do_syscall_WRK implementation for that architecture. Implement
and adjust line numbers in .exp file to make it PASS.

Add (minimal) manual page for valgrind-di-server.

This adds a minimal man page for the experimental valgrind-di-server
so that we have manual pages for all (auxiliary) programs we install.

PPC64, fix test_isa_3_0_other.c test

Valgrind ppc64 test_isa_3_0_other test will attempt to display
all of the bits of the XER as part of the test case results.
The tests have no existing logic to clear those bits, so this can
pick up straggling values that cascade into a testcase failure.
This adds some code to correct this in two directions;
    - Print only the bits that are expected by the tests.  This
    is currently just the OV and OV32 bits.
    - print all of the bits when run under higher verbosity levels.

Bugzilla 406198 - none/tests/ppc64/test_isa_3_0_other test sporadically
                  including CA bit in output

Patch submitted by  Will Schmidt <will_schmidt@vnet.ibm.com>
Patch reviewed, committed by: Carl Love <cel@us.ibm.com>

Bug 404843 - s390x: backtrace sometimes ends prematurely.

On s390x-linux, adds CFI based unwinding for %f0..%f7, since these are sometimes
used by gcc >= 8.0 to spill integer register values in leaf functions.  Hence the
lack of unwinding them was causing unwind failures on this platform.

PPC64, patch to test case issues reported in bugzilla 401827 and 401828.

This corrects a valgrind instruction emulation issue revealed by
a GCC change.
The xscvdpsp,xscvdpspn,xscvdpuxws instructions each convert
double precision values to single precision values, and write
the results into bits 0-32 of the 128 bit target register.
To get the value into the normal position for a scalar register
the result needed to be right-shifted 32 bits, so gcc always
did that.
It was determined that hardware also always did that, so the (redundant)
gcc shift was removed.
This exposed an issue because valgrind was only writing the result to
bits 0-31 of the target register.

This patch updates the emulation to write the result to both of the involved
32-bit fields.

VEX/priv/guest_ppc_toIR.c:
  - rearrange ops in dis_vx_conv to update more portions of the target
    register with copies of the result.   xscvdpsp,xscvdpspn,xscvdpuxws

none/tests/ppc64/test_isa_2_06_part1.c
  - update res32 checking to explicitly include fcfids and fcfidus in the
    32-bit result grouping.

none/tests/ppc64/test_isa_2_07_part2.c
  - correct NULL initializer for logic_tests definition

[*1] - GCC change referenced:
    2017-09-26  Michael Meissner  <meissner@linux.vnet.ibm.com>
            * config/rs6000/rs6000.md (movsi_from_sf): Adjust code to
              eliminate doing a 32-bit shift right or vector extract after
              doing  XSCVDPSPN.

patch submitted by:   Will Schmidt <will_schmidt@vnet.ibm.com>
reviewed, committed by:  Carl Love <cel@us.ibm.com>

DHAT: when the run ends, print a how-to-view-the-profile hint message.  n-i-bz.

The aim is to make it zero-effort for users to view the profile after
a run.  The printed message is as follows:

  To view the resulting profile, open
    file:///path/to/valgrind/installation/lib/valgrind/dh_view.html
  in a web browser, click on "Load..." and then select the file
    /path/to/dhat.out.12345
  Scroll to the end the displayed page to see a short
  explanation of some of the abbreviations used in the page.

This patch adds printing of the message, then filters it out in
dhat/tests/filter_stderr, and updates the .stderr.exp files to
remove blank lines.

mips32: pass correct syscall value to kernel in case of __NR_syscall

The syscall number has to be put in register v0 before call into the kernel.
This was omitted when system call is __NR_syscall (and when the syscall
argument is the system call number of interest).

Patch by Nikola Milutinovic.

Support arm64 core dump

Fixes BZ #405722.
Implements coredump-elf.c fill_prstatus()
and fill_fpu () for VGP_arm64_linux.

Update NEWS and docs/internals/3_14_BUGSTATUS.txt to reflect current bug-fix status.

Announce fix 405782  "VEX temporary storage exhausted" when attempting to debug slic3r-pe

This bug was fixed by various optimisations done by Julian.

mips: get rid of format and implicit-fallthrough warnings

Indicate when the fall through from the previous case label is intentional.
Fix format warnings related to arguments in printf calls.

update NEWS about fixed KDE #400975

KDE #400975 has been fixed with commit
e61d13087096139024788393218367572751b4b6

mips: code refactoring (NFC)

Code in VEX/priv/guest_mips_toIR.c is notably refactored.
DSP ASE dissasembly has been put in a separate file: guest_mipsdsp_toIR.c.

Patch by Aleksandar Rikalo.

Use ULong instead of unsigned long in s390_irgen_EX_SS.

ovl was defined as an unsigned long. This would cause warnings from gcc:

  guest_s390_toIR.c:195:30: warning: right shift count >= width of type
  [-Wshift-count-overflow]

when building on 32bit arches, or building a 32bit secondary arch.

Fix this by defining ovl as ULong which is always guaranteed 64bit.

Use gcc -Wimplicit-fallthrough=2 by default if available

GCC 7 instroduced -Wimplicit-fallthrough
https://developers.redhat.com/blog/2017/03/10/wimplicit-fallthrough-in-gcc-7/

It caught a couple of bugs, but it does need a bit of extra comments to
explain when a switch case statement fall-through is deliberate. Luckily
with -Wimplicit-fallthrough=2 various existing comments already do that.
I have fixed the bugs, but adding explicit break statements where
necessary and added comments where the fall-through was correct.

https://bugs.kde.org/show_bug.cgi?id=405430

mips: use local labels for do_acasW()

use local labels for do_acasW() to avoid defining symbols multiple times
when the function gets inlined.

It fixes assembler error reported in KDE #400164.

update .gitignore with amd64 tests f16c and rdrand

Add
  /none/tests/amd64/f16c
  /none/tests/amd64/rdrand

to .gitignore.

mips: if revision is specified, use it for compilation

If user has specified a particular MIPS revision in configure line,
do not interfere with this (i.e. skip setting FLAG_M32/FLAG_M64).

Related KDE issues #402123 and #400975.

Patch by Stefan Maksimovic.

mips: add a comment about decoding LX on Cavium

Interpret LX as a Cavium instruction, otherwise try decoding it as a DSP
instruction.
The fallthrough is deliberate.

Related to KDE #405430.

PPC64, fix for vrlwnm, vrlwmi, vrldrm, vrldmi instructions.

Fixes the case where the specified end bit is less then the start bit.

Valgrind bug 405734

PPC64, fix output for xvcvdpsp instruction.

The instruction should write the output to the upper and lower 32-bit
halfs of the results.

Valgrind bugzilla 405733.

PPC64, The function _get_maxmin_fp_NaN does not handle the case of QNaN, SNaN correctly.

This patch fixes Valgrind to handle the case of QNaN, SNaN input the same
as the HW handles it.

Valgrind bug 405365.

PPC64, instructions xvcvdpsxws, xvcvdpuxws do not handle over/underflow, NaN correctly

The instructions are not checking for overflow, underflow, NaN and setting
the output correctly.

Valgrind bugzilla 405363

PPC64, fix for vmsummbm instruction.

The instruction needs to have the 32-bit "lane" values chopped to 32-bits.
The current lane implementation is not doing the chopping.  Need to
explicitly do the chop and add.

Valgrind bug 405362

PPC64, fix implementation of xvcvsxdsp and xvcvuxddp instructions.

Instructions need to write result to upper and lower 32-bit half of the
64-bit result.

This is a fix for Valgrind bug 405356.

drd: Fix an integer overflow in the stack margin calculation

update NEWS about the fix for KDE #405458

KDE #405458 has been fixed in the previous commit

  commit 029f1196fcafb49183c87c53e3a7febc762a0dc8
  Author: Petar Jovanovic <mips32r2@gmail.com>
  Date:   Mon Mar 18 16:47:50 2019 +0100

  mips: correct order of function arguments for mkFormVEC

mips: correct order of function arguments for mkFormVEC

Vectors wt and ws were incorrectly received in mkFormVEC().
Issue spotted by Mark Wielaard and reported as KDE #405458.

amd64: Implement RDRAND, VCVTPH2PS and VCVTPS2PH.

Bug 398870 - Please add support for instruction vcvtps2ph
Bug 353370 - RDRAND amd64->IR: unhandled instruction bytes: 0x48 0xF 0xC7 0xF0

This commit implements:

* amd64 RDRAND instruction, on hosts that have it.

* amd64 VCVTPH2PS and VCVTPS2PH, on hosts that have it.

  The presence/absence of these on the host is now reflected in the CPUID
  results returned to the guest.  So code that tests for these features in
  CPUID and acts accordingly should "just work".

* New test cases, none/tests/amd64/rdrand and none/tests/amd64/f16c.  These
  are built if the host's assembler can handle them, in the usual way.

Fix Bug 404638 - Add VG_(replaceIndexXA)

Based on a patch from Łukasz Marek.

Note that this function differs from:
   *(T*)VG_(indexXA)(arr, index) = new_value;
as the function will mark the array as unsorted.

Note that this function is currently unused in the current valgrind code basis,
but it is useful for tools outside of valgrind tree.

Bug 405403 - s390x: Allow using disInstr_S390 on little-endian hosts

Certain projects, e.g. https://angr.io, use VEX as an intermediate
representation for the binary code analysis. In order to make it
possible to use them to analyze S/390 code on Intel, this patch
resolves the following issues in the disassembler:

- Bit fields, which are used to describe instruction formats, map to
  different bits on different hosts. This patch replaces them with
  macros, e.g. SS.l bit field becomes SS_l macro. Most bit field usages
  are replaced using the following perl script:

    perl -p -i \
         -e 's/\(&ovl\.value\)/&ovl/g;' \
         -e 's/ovl\.value/ovl/g;' \
         -e 's/ovl\.fmt\.([a-zA-Z\d_]+)\.([a-z\d]+)/$1_$2(ovl)/g' \
         priv/guest_s390_toIR.c

  Since after that there are no more structs, #pragma pack is also
  removed.

- Instructions are loaded from memory as words, which behaves
  differently depending on host endianness. Such loads are replaced by
  assembly of words from separately loaded bytes. This affects regular
  disassembly functions, and also s390_irgen_EXRL(), which loads
  last_execute_target this way.

- disInstr_S390() explicitly prohibits little-endian hosts with an
  assert, which is removed in this patch.

Signed-off-by: Ilya Leoshkevich <iii@linux.ibm.com>

Finetune arch_hwcaps() in none/tests/libvex_test.c

The libvexmultiarch_test failed on s390, since VEX was configured for MIPS64
with 32bit FPUs. Modify arch_hwcaps() to a realist case with 64bit FPUs.

This fixes KDE #402351.

Document 403123 as fixed

Suppress FSGSBASE flag from cpuid results

We don't support {rd,wr}{fs,gs}base so we shouldn't say we do.

Bug 399287 - amd64 front end: Illegal Instruction vcmptrueps.  Add test cases.

Bug 399287 - amd64 front end: Illegal Instruction vcmptrueps.  Fix, but no test cases.

Rename gettid() to gettid_sys() in gdbserver_tests.

glibc might defined gettid() itself through unistd.h:
https://sourceware.org/bugzilla/show_bug.cgi?id=6399

Rename to gettid_sys() so we don't clash with the glibc definition.

VEX/auxprogs/genoffsets.c: Add cast to my_offsetof.  n-i-bz.

Clang/LLVM trips over my_offsetof in VEX/auxprogs/genoffsets.c.  See LLVM
PR 40890 for details (https://bugs.llvm.org/show_bug.cgi?id=40890).

Now, it's a Clang bug that Clang exits on an assertion failure rather than
emits a diagnostic, but the previous my_offsetof expression is a pointer,
not an integer.  Add a cast as done in other definitions of offsetof in
the tree.  Patch from Ed Maste <emaste@freebsd.org>.

Add missing documentation file from EXTRA_DIST

Ensure this documentation file is included in tarball.

Fixes: 9f3d49a ("Create 3.10.0 section in NEWS, add (first draft of) README_DEVELOPERS_processes")
Signed-off-by: Rhys Kidd <rhyskidd@gmail.com>

macOS: Don't duplicate -fno-stack-protector

Since f38d96d -fno-stack-protector has been added to $(AM_CFLAGS_BASE) on all
platforms, if the compiler supports it. Accordingly, there's no need to still add
this a second time specifically for macOS.

Fixes: f38d96d ("Add -Wformat -Wformat-security to the list of compile flags.")
Signed-off-by: Rhys Kidd <rhyskidd@gmail.com>

config: Conditionalize -finline-functions on compiler support

Certain clang compiler versions do not support -finline-functions, so only apply
this compiler option conditionally if supported.

Warnings with Apple LLVM version 8.0.0 (clang-800.0.42.1), based on upstream clang 3.9.0:

  clang: warning: optimization flag '-finline-functions' is not supported
  clang: warning: argument unused during compilation: '-finline-functions'

Fixes: 7dd9a7f ("Add -finline-functions to standard build flags, so gcc will
                 consider all functions as candidates for inlining.")

Signed-off-by: Rhys Kidd <rhyskidd@gmail.com>

Makefile.am: Consistent indent and align

This approach is utilized consistently throughout the file, and would make
alphabetical reordering easier. Adopt it here as well.

Signed-off-by: Rhys Kidd <rhyskidd@gmail.com>

config: Set automake options consistenly in one location

Avoid use of two manners of setting automake configuration options,
and consolidate them into AM_INIT_AUTOMAKE macro.

Signed-off-by: Rhys Kidd <rhyskidd@gmail.com>

config: remove unrequired AC_HEADER_STDC

Autoconf says:
"This macro is obsolescent, as current systems have conforming
header files. New programs need not use this macro".

Was previously required to ensure the system has C header files conforming
to ANSI C89 (ISO C90). Specifically, this macro checks for stdlib.h,
stdarg.h, string.h, and float.h.

This autoconf option was used to provide conditional fallback support
via defined STDC_HEADERS.

valgrind does not utilize conditional fallback support so, so this macro
is both obsolete and unused, so let's drop it.

Signed-off-by: Rhys Kidd <rhyskidd@gmail.com>

Add a 3_14_BUGSTATUS.txt file and add to it all bugs reported since 3.14 was release.

At least, the bugs are post-triaged ones, so some have been removed.

bb_to_IR(): increase assertion limits on the maximum size of self-checking translations.  n-i-bz.

modify massif/tests/mmapunmap.vgtest to comply with glibc change

The change in the glibc version (2.27 -> 2.28) results in one additional
function call being present in the backtrace for mips64, which leads to the
line to be checked to be out of bounds.

Changed the post line in mmapunmap.vgtest to work around this.

This fixes massif/tests/mmapunmap failure on mips64.

Patch by Stefan Maksimovic.

Bug 405079 - unhandled ppc64le-linux syscall: 131 (quotactl)

quotactl is really a "generic" linux syscall that just happened to not
have been hooked up for ppc64le. Add it to syswrap-ppc64-linux.c.

Adjust the built-in profiler so that it can try to count host insns as well as guest insns.  n-i-bz.

Fix another format string signedness warning, arm64-linux only.  n-i-bz.

memcheck powerpc subfe x, x, x initializes x to 0 or -1 based on CA

GCC might use subfe x, x, x to initialize x to 0 or -1, based on
whether the carry flag is set. This happens in some cases when g++
compiles resetting a unique_ptr. The "trick" used by the compiler is
that it can AND a pointer with the register x (now 0x0 or 0xffffffff)
to set something to NULL or to the given pointer.

subfe is implemented as rD = (log not)rA + rB + XER[CA]
if we instead implement it as rD = rB - rA - (XER[CA] ^ 1)
then memcheck can see that rB and Ra cancel each other out if they
are the same.

https://bugs.kde.org/show_bug.cgi?id=404054

Fix missed changes from Rename some int<->fp conversion IROps patch

The previous commit 6b16f0e2a0427f57fb5dc76cbe9177ee35f997ab dated
Sat Jan 26 17:38:01 2019 by Julian Seward <jseward@acm.org> renamed some of
the int<->fp conversion Iops to add a trailing _DEP.  The patch missed
renaming two of the Iops.  This patch renames the missed Iops.

Make the DHAT viewer components be copied into the distribution tarball.  Followup to 441bfc5f51c7f5f80cc6491d23cbe2dc711d191f (dhat overhaul).

Make the DHAT viewer components be copied into the install tree.  Followup to 441bfc5f51c7f5f80cc6491d23cbe2dc711d191f (dhat overhaul).

Fix "make post-regtest-checks" after 441bfc5f51c7f5f80cc6491d23cbe2dc711d191f (dhat overhaul).