[flang-commits] [flang] [llvm] [flang] Enumeration Type: (PR 3/5) Intrinsics + I/O + Modules (PR #193235)

[via flang-commits]

MattPD wrote:

Thank you for working through this. You addressed everything I raised, and we agreed to handle the out-of-range diagnostic and its test in PR 4/5.

One question you raised still appears unanswered. You asked whether enumeration values should be allowed in unformatted I/O using their underlying integer ordinals, or whether a separate issue should track that decision. The code in `check-io.cpp` that examines enumeration components enforces the current restriction, so the answer decides how much of that code remains.

@eugeneepshteyn, could you confirm whether that needs settling here or can move to a follow-up issue? Approval is your call as the area reviewer. I have no objection to this landing once that question is answered.


https://github.com/llvm/llvm-project/pull/193235