RFR: 8294438: Fix misleading-indentation warnings in hotspot [v3]
David Holmes
dholmes at openjdk.org
Thu Oct 20 07:34:00 UTC 2022
On Thu, 20 Oct 2022 07:14:27 GMT, Aleksey Shipilev <shade at openjdk.org> wrote:
>> Aleksey Shipilev has updated the pull request with a new target base due to a merge or a rebase. The pull request now contains five commits:
>>
>> - Merge branch 'master' into JDK-8294438-misleading-indentation
>> - Merge branch 'master' into JDK-8294438-misleading-indentation
>> - Merge branch 'master' into JDK-8294438-misleading-indentation
>> - Also javaClasses.cpp
>> - Fix
>
> Thanks!
@shipilev this has broken our linux aarch64 builds!
[2022-10-20T07:26:59,542Z] workspace/open/src/hotspot/cpu/aarch64/assembler_aarch64.cpp: In member function 'void Address::lea(MacroAssembler*, Register) const':
[2022-10-20T07:26:59,542Z] workspace/open/src/hotspot/cpu/aarch64/assembler_aarch64.cpp:138:5: error: this 'else' clause does not guard... [-Werror=misleading-indentation]
[2022-10-20T07:26:59,542Z] 138 | else
[2022-10-20T07:26:59,542Z] | ^~~~
[2022-10-20T07:26:59,542Z] workspace/open/src/hotspot/cpu/aarch64/assembler_aarch64.cpp:140:7: note: ...this statement, but the latter is misleadingly indented as if it were guarded by the 'else'
[2022-10-20T07:26:59,542Z] 140 | break;
[2022-10-20T07:26:59,542Z] | ^~~~~
-------------
PR: https://git.openjdk.org/jdk/pull/10444
More information about the shenandoah-dev
mailing list