projects / platform / upstream / llvm.git / commit
? search:
summary | shortlog | log | commit | commitdiff | tree
(parent: 53d8687) | patch
[flang][runtime] ensure character compares to blank are unsigned
authorJean Perier <jperier@nvidia.com>
Tue, 13 Sep 2022 08:39:33 +0000 (10:39 +0200)
committerJean Perier <jperier@nvidia.com>
Tue, 13 Sep 2022 08:42:53 +0000 (10:42 +0200)
commit2694234c2982c57b9442ad1a1b4ffb2100f4ff34
treeabb51762a4613fe5bc3b0922a5305c0df238abd9tree | snapshot
parent53d8687a13e76b5a387e8df59ae231ab53ab9279commit | diff
[flang][runtime] ensure character compares to blank are unsigned

CompareToBlankPadding was doing signed compare on architecture where
`char` is signed. This caused `'abc'//char(128) > 'abc'` to evaluate
to false at runtime instead of true.

Differential Revision: https://reviews.llvm.org/D133693
flang/runtime/character.cpp diff | blob | history
flang/unittests/Runtime/CharacterTest.cpp diff | blob | history
Domain: System / Toolchain;