Update find-in-function method to be more vague about the leas it finds when precise-switch-statement finding is impossible

Issue was related to intermediate set of the compare register.
Leveraged backup_until's stopif argument to stop backup until sooner.

Note: as indicated in code comment, single register/register move
is common and could be used to improve accuracy.

1) FII was finding the wrong cmp for switch tables.
Removed the blind cmp and forced the cmp to use the proper
register or not find a cmp.  Also changed the limit for a table
size when no cmp is found from 2^32 to 255.

2) Fixed an issue with zipr dollop layout where the size
to layout 1 instruction + a trampoline was bigger than laying
out two+ full dollops (because both dollops were short), but
the dollops are disallowed from being coalesced.  The worst-size
detector needed to account for non-coalescable dollops.

Deal with prefixes on jmp/jcc/call instructions that might change the insn length, fix syntax for x86

Deal with prefixes on jmp/jcc/call instructions that might change the insn length, fix syntax for arm

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.