[flang-commits] [flang] [llvm] [CodeGen] Take the executable stack from a module flag (PR #215152)

[Sairudra More via flang-commits]

Saieiei wrote:

Thanks, the single `0 -> 1` case is covered now. Could r`equestExecutableStack()` also skip `Require` entries when looking for the executable-stack definition and search all `llvm.module_flags` ops before deciding to append?

A same-key `Require` entry can currently be overwritten, while an existing `executable-stack = 0` in a later flags op can be missed, leaving duplicate non-`Require` definitions during translation. Tests for those two cases would cover the remaining edge cases.

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