On Wed, 13 Mar 2024 11:18:20 GMT, Magnus Ihse Bursie <i...@openjdk.org> wrote:

> As part of the ongoing effort to enable jcheck whitespace checking to all 
> text files, it is now time to address assembly (.S) files. The hotspot 
> assembly files were fixed as part of the hotspot mapfile removal, so only a 
> single incorrect jdk library remains.
> 
> The main issue with `libjsvml` was inconsistent line starts, sometimes using 
> tabs. I used the `expand` unix command line tool to replace these with spaces.

src/jdk.incubator.vector/linux/native/libjsvml/jsvml_d_acos_linux_x86.S line 37:

> 35:         .text
> 36: # mark_begin;
> 37:        .align    16,0x90

.align seems to ironically not be aligned with the rest of the directives

-------------

PR Review Comment: https://git.openjdk.org/jdk/pull/18268#discussion_r1523048988

Reply via email to