tree 42c9f97ecac983ccf4e041036cc8d15ecc58098d
parent 4922e10b45a8cb948d6f3ef7b9414eabd494b4dc
author Peter Steinfeld <psteinfeld@nvidia.com> 1615252634 -0800
committer Copybara-Service <copybara-worker@google.com> 1616051875 -0700

[flang] Sort symbols by creation order

We have a "<" operator defined on the type semantics::Symbol that's based on
the symbols' locations in the cooked character stream.  This is potentially
problematic when comparing symbols from .mod files when the cooked character
streams themselves might be allocated to varying memory locations.

This change fixes that by using the order in which symbols are created as the
basis for the "<" operator.  Thanks to Tim and Peter for consultation on the
necessity of doing this and the idea for what to use as the basis of the sort.

This change in the "<" operator changed the expected results for three of the
tests.  I manually inspected the new results, and they look OK to me.  The
differences in data05.f90 and typeinfo01.f90 are entirely the order, offsets,
and sizes of the derived type components.  The changes in resolve102.f90 are
due to the new, different "<" operator used for sorting.

Differential Revision: https://reviews.llvm.org/D98225

GitOrigin-RevId: 63200a058e6d5f275204154a16723e8f169a317a