[PATCH] D101355: Fix a bug in the character runtime

Diana Picus via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Apr 27 04:31:09 PDT 2021


rovka created this revision.
rovka added a reviewer: klausler.
rovka requested review of this revision.
Herald added a project: LLVM.
Herald added a subscriber: llvm-commits.

The number of bytes copied in CopyAndPad should depend on the size of
the type being copied, not on its shift value (which in the case of char
is 0, leading to no bytes at all being copied).

Add unit tests for CharacterMin and CharacterMax, which exercise this
code path.


Repository:
  rG LLVM Github Monorepo

https://reviews.llvm.org/D101355

Files:
  flang/runtime/character.cpp
  flang/unittests/RuntimeGTest/CharacterTest.cpp

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D101355.340794.patch
Type: text/x-patch
Size: 3890 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20210427/d085506f/attachment.bin>


More information about the llvm-commits mailing list