[Mlir-commits] [mlir] [flang] [mlir][openacc] Add legalize data pass for compute operation (PR #80351)

Valentin Clement バレンタイン クレメン llvmlistbot at llvm.org
Mon Feb 5 08:38:38 PST 2024


Valentin Clement =?utf-8?b?KOODkOODrOODsw=?Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/80351 at github.com>


clementval wrote:

> Looks great! Implementation looks much cleaner than I imagined.
> 
> I do think that the ssa uses for private/reduction/firstprivate operations should also be replaced. This means that the logic would also need to apply to acc.loop in addition to the compute constructs. This should be added also under an option just like you did for hostToDevice.
> 
> I would say this is OK to submit as-is - and we can make the improvement when it is needed. Nice work! Thank you for getting this done :)

I'm gonna make a follow up PR for private/reduction

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


More information about the Mlir-commits mailing list