[PATCH] D110155: [OpenCL] Allow optional __generic in __remove_address_space utility

2021-12-27 Thread Justas Janickas via Phabricator via cfe-commits
Topotuna closed this revision. Topotuna added a comment. Closed by commit: rGb5fef6dbfd6b4c84d61155fd2221064bcc1ecdcf CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110155/new/ https://reviews.llvm.org/D110155 __

[PATCH] D110155: [OpenCL] Allow optional __generic in __remove_address_space utility

2021-12-02 Thread Anastasia Stulova via Phabricator via cfe-commits
Anastasia accepted this revision. Anastasia added a comment. This revision is now accepted and ready to land. LGTM! Thanks CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110155/new/ https://reviews.llvm.org/D110155 ___ cfe-commits mailing list

[PATCH] D110155: [OpenCL] Allow optional __generic in __remove_address_space utility

2021-11-30 Thread Justas Janickas via Phabricator via cfe-commits
Topotuna updated this revision to Diff 390617. Topotuna added a comment. Run line argument reordering CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110155/new/ https://reviews.llvm.org/D110155 Files: clang/lib/Headers/opencl-c-base.h clang/test/CodeGenOpenCLCXX/remove-address-space

[PATCH] D110155: [OpenCL] Allow optional __generic in __remove_address_space utility

2021-11-12 Thread Justas Janickas via Phabricator via cfe-commits
Topotuna updated this revision to Diff 386773. Topotuna added a comment. Herald added a subscriber: Naghasan. Run line `-triple` is now set explicitly. Test moved from `CodeGenOpenCLCXX` directory to `SemaOpenCLCXX` CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110155/new/ https://revi

[PATCH] D110155: [OpenCL] Allow optional __generic in __remove_address_space utility

2021-11-09 Thread Justas Janickas via Phabricator via cfe-commits
Topotuna added a comment. In D110155#3117786 , @carlosgalvezp wrote: > Hi! I believe this commit broke a number of CI jobs, would you be able to > look into it? > > https://lab.llvm.org/buildbot/#/builders/52/builds/12263 Hi, I am aware it broke some o

[PATCH] D110155: [OpenCL] Allow optional __generic in __remove_address_space utility

2021-11-09 Thread Carlos Galvez via Phabricator via cfe-commits
carlosgalvezp added a comment. Hi! I believe this commit broke a number of CI jobs, would you be able to look into it? https://lab.llvm.org/buildbot/#/builders/52/builds/12263 Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110155/new/ https://revi

[PATCH] D110155: [OpenCL] Allow optional __generic in __remove_address_space utility

2021-11-09 Thread Justas Janickas via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rG81081daef029: [OpenCL] Allow optional __generic in __remove_address_space utility (authored by Topotuna). Repository: rG LLVM Github Monorepo CHA

[PATCH] D110155: [OpenCL] Allow optional __generic in __remove_address_space utility

2021-09-21 Thread Anastasia Stulova via Phabricator via cfe-commits
Anastasia accepted this revision. Anastasia added a comment. This revision is now accepted and ready to land. LGTM! Thanks Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D110155/new/ https://reviews.llvm.org/D110155 _

[PATCH] D110155: [OpenCL] Allow optional __generic in __remove_address_space utility

2021-09-21 Thread Justas Janickas via Phabricator via cfe-commits
Topotuna created this revision. Topotuna added a reviewer: Anastasia. Herald added subscribers: ldrumm, yaxunl. Topotuna requested review of this revision. Herald added a project: clang. Herald added a subscriber: cfe-commits. Clang builtin utility `__remove_address_space` now works if generic add