[flang-commits] [flang] [mlir] [Flang][OpenMP] Fix to resolve the crash with SIMD aligned clause. (PR #150612)

[Tom Eccles via flang-commits]

tblah wrote:

My understanding from the discussion today is that it is important that we do not generate errors for non-power-of-2 alignments, however it would be perfectly standards compliant to drop any alignment information which is not currently supported by the backend.

My suggestion would be to add the warning to flang semantics for the reasons I already described. I think in light of the discussion today, it isn't too important that we add something to the MLIR level warning other users about this, because what we are doing is still standards compliant.

But I don't feel that strongly about the warning placement in semantics. There should be a warning _somewhere_, then (as an implementation detail) we should drop any alignment numbers that cannot be passed on to the backend.

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