about summary refs log tree commit diff
path: root/sysdeps/x86_64/fpu/s_fminf.S
diff options
context:
space:
mode:
authorNoah Goldstein <goldstein.w.n@gmail.com>2021-09-21 18:45:03 -0500
committerNoah Goldstein <goldstein.w.n@gmail.com>2021-10-12 12:02:12 -0500
commit1bd8b8d58fc9967cc073d2c13bfb6befefca2faa (patch)
tree056a0dcdfc1c587b9e4fc279ce8573bbcb0ee218 /sysdeps/x86_64/fpu/s_fminf.S
parent8faa1e04493f23b16f473d21a3a5bc49b781ed2a (diff)
downloadglibc-1bd8b8d58fc9967cc073d2c13bfb6befefca2faa.tar.gz
glibc-1bd8b8d58fc9967cc073d2c13bfb6befefca2faa.tar.xz
glibc-1bd8b8d58fc9967cc073d2c13bfb6befefca2faa.zip
x86: Optimize memcmp-evex-movbe.S for frontend behavior and size
No bug.

The frontend optimizations are to:
1. Reorganize logically connected basic blocks so they are either in
   the same cache line or adjacent cache lines.
2. Avoid cases when basic blocks unnecissarily cross cache lines.
3. Try and 32 byte align any basic blocks possible without sacrificing
   code size. Smaller / Less hot basic blocks are used for this.

Overall code size shrunk by 168 bytes. This should make up for any
extra costs due to aligning to 64 bytes.

In general performance before deviated a great deal dependending on
whether entry alignment % 64 was 0, 16, 32, or 48. These changes
essentially make it so that the current implementation is at least
equal to the best alignment of the original for any arguments.

The only additional optimization is in the page cross case. Branch on
equals case was removed from the size == [4, 7] case. As well the [4,
7] and [2, 3] case where swapped as [4, 7] is likely a more hot
argument size.

test-memcmp and test-wmemcmp are both passing.
Diffstat (limited to 'sysdeps/x86_64/fpu/s_fminf.S')
0 files changed, 0 insertions, 0 deletions