[clang] [flang] [llvm] [flang][OpenMP] Add semantic check for declare target (PR #72770)

2023-11-20 Thread via cfe-commits
https://github.com/agozillon approved this pull request. LGTM, aside from Sergio's Nits! Might be worth asking someone from the Clang team to have a look at the Clang changes if you're a little unsure on them, but if you're happy with them can disregard doing that!

[clang] [flang] [llvm] [flang][OpenMP] Add semantic check for declare target (PR #72770)

2023-11-20 Thread Sergio Afonso via cfe-commits
https://github.com/skatrak commented: Thank you Shraiysh, I just have a couple of nits. https://github.com/llvm/llvm-project/pull/72770 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits