[PATCH] D155978: [SPIRV] Add SPIR-V logical triple.

2023-09-06 Thread Michal Paszkowski via Phabricator via cfe-commits
mpaszkowski added a comment. In D155978#4639499 , @Keenuts wrote: > In D155978#4639094 , @mpaszkowski > wrote: > >> @Keenuts Hi Nathan, thanks for the patch! I agree with your approach and I >> think that this s

[PATCH] D155978: [SPIRV] Add SPIR-V logical triple.

2023-09-05 Thread Michal Paszkowski via Phabricator via cfe-commits
mpaszkowski accepted this revision. mpaszkowski added a comment. This revision is now accepted and ready to land. @Keenuts Hi Nathan, thanks for the patch! I agree with your approach and I think that this solution despite being a "hack" seems to be the most straightforward. Eventual differences

[PATCH] D125679: [Clang] Added options for integrated backend only used for SPIR-V for now

2022-05-18 Thread Michal Paszkowski via Phabricator via cfe-commits
mpaszkowski accepted this revision. mpaszkowski added a comment. This revision is now accepted and ready to land. The typos noticed by Ilia need to be fixed, but apart from this the patch LGTM! Thank you Anastasia! CHANGES SINCE LAST ACTION https://reviews.llvm.org/D125679/new/ https://revie