diff --git a/flang/runtime/descriptor.cpp b/flang/runtime/descriptor.cpp --- a/flang/runtime/descriptor.cpp +++ b/flang/runtime/descriptor.cpp @@ -31,9 +31,23 @@ int rank, const SubscriptValue *extent, ISO::CFI_attribute_t attribute, bool addendum) { Terminator terminator{__FILE__, __LINE__}; + // Subtle: the standard CFI_establish() function doesn't allow a zero + // elem_len argument in cases where elem_len is not ignored; and when it + // returns an error code (CFI_INVALID_ELEM_LEN in this case), it must not + // modify the descriptor. That design makes sense, maybe, for actual + // C interoperability, but we need to work around it here. A zero + // incoming element length is replaced by 4 so that it will be valid + // for all CHARACTER kinds. + std::size_t workaroundElemLen{elementBytes ? elementBytes : 4}; RUNTIME_CHECK(terminator, - ISO::CFI_establish(&raw_, p, attribute, t.raw(), elementBytes, rank, + ISO::CFI_establish(&raw_, p, attribute, t.raw(), workaroundElemLen, rank, extent) == CFI_SUCCESS); + if (elementBytes != ElementBytes()) { + raw_.elem_len = elementBytes; + for (int j{0}; j < rank; ++j) { + GetDimension(j).SetByteStride(0); + } + } raw_.f18Addendum = addendum; DescriptorAddendum *a{Addendum()}; RUNTIME_CHECK(terminator, addendum == (a != nullptr));