Deal with fallthroughs to unpinned IBTAs

See merge request !15

Previously, dollop coalescing would allow unpinned IBTAs disobey
the IBTA alignment restrictions.  To deal with alignment,
the dollop coalescing was disallowed if an unpin to attempt to coalesce
is in a different function than the code that the unpin would would be
coalesced to.

Move-globals speedup

See merge request !14

New feature allows sizer to specify the alignment of unpinned
instructions (i.e., instructions with a pin, but no pin address).

Feature applies only to instructions that are _not_ the fallthrough
of other instructions (e.g., function pointers)

Also, code cleanups (removing commented-out code, spacing, tabbing)

This reverts commit ed60f36d.

May need to revisit this because pin-alignment needs to be
a function of the dollop manager, instead of the hacky method it is now.

Debugging code allows for one to binary-search the unpins.

But a .dynsym, .got, etc. entry can be duplicated in it's own section
or other sections.  Unpinning has to handle unpinning an instruction
and deal with updating all tables properly.  Bug was found in handling
of the unpin for .dynsym in this regard.  This patch fixes that bug.

Rarely executed check for table-table entry encoding returned
that mismatching encodings are OK, but later ::extend() asserted
they had to be the same.  Fixed check to return that mismatches
result in failed extend instead of abort.

In particulate, an alternate version of switch tables
was being mis-detected by FII.  In particular, switch
tables were assumed to be size=4, but we found size!=4
in sqlite (in particular, size=8).  And, I know windows
binaries have size=1,2.

Also fixed an issue where lea reg1, [reg2+reg3] assumed reg2/reg3
had values (i.e., reg2 is the base address of a switch table),
when reg3 might have the value.  Fixed the assumption by
tracing both reg2+reg3.

Some minor code cleanups.