[Mlir-commits] [mlir] [mlir][SPIRV] Fix lookup logic `spirv.target_env` for `gpu.module` (PR #147262)

[Igor Wodiany]

https://github.com/IgWod-IMG commented:

A rather naive question, as I'm not familiar with the GPU dialect. Is it possible for the target env to be present in both `"targets"` and as an op attribute at the same time? If so, does the look-up order matter? I’d hope the same target env is returned either way, but hypothetically if there is a different env in `"targets"` and in the op, I wonder which one should take a precedence?

Also, it'd be nice to have a test for the change. Probably in GPU to SPIR-V conversion directory?

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