[PATCH] D78970: [CUDA][HIP] Fix ambiguity of new operator

2020-04-28 Thread Yaxun Liu via Phabricator via cfe-commits
yaxunl closed this revision. yaxunl added a comment. committed by 55bcb96f3154808bcb5afc3fb46d8e00bf1db847 CHANGES SINCE LAST ACTION https://reviews.llvm.org/D78970/new/ https://reviews.llvm.org/D78970

[PATCH] D78970: [CUDA][HIP] Fix ambiguity of new operator

2020-04-28 Thread Artem Belevich via Phabricator via cfe-commits
tra accepted this revision. tra added a comment. OK. Let's give it a try. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D78970/new/ https://reviews.llvm.org/D78970 ___ cfe-commits mailing list cfe-commits@lists.llvm.org

[PATCH] D78970: [CUDA][HIP] Fix ambiguity of new operator

2020-04-27 Thread John McCall via Phabricator via cfe-commits
rjmccall accepted this revision. rjmccall added a comment. This revision is now accepted and ready to land. This really has nothing to do with the `new` operator specifically, but it looks like a good fix. CHANGES SINCE LAST ACTION https://reviews.llvm.org/D78970/new/

[PATCH] D78970: [CUDA][HIP] Fix ambiguity of new operator

2020-04-27 Thread Yaxun Liu via Phabricator via cfe-commits
yaxunl created this revision. yaxunl added reviewers: tra, rjmccall. https://reviews.llvm.org/D77954 caused a regression about ambiguity of new operator in file scope. This patch recovered the previous behavior for comparison without a caller. This is a workaround. For real fix we need D71227