[clang] [SYCL] AST support for SYCL kernel entry point functions. (PR #122379)
Tom Honermann via cfe-commits
cfe-commits at lists.llvm.org
Wed Jan 22 10:11:51 PST 2025
tahonermann wrote:
> No idea why I can't respond to the individual threads but...
@erichkeane, in order to reduce notifications, I replied to your comments as a review. You should generally be able to reply to them individually or as part of a batched review from the "Files changed" tab, but GitHub doesn't permit such replies from the "Conversation" tab (which is weird and seems dumb). However, if the comments are considered "outdated" because of new commits that changed the code the thread was anchored too, then such comments appear to become orphaned; they still appear in the "Conversation" tab, but not in the "Filed changed" tab and therefore can't be replied to. GitHub is a less than perfect tool, IMHO of course.
> make format happy, then LGTM.
Thank you. I addressed the non-dumb clang-format complaints. I'll merge once @bader is happy with the changes.
https://github.com/llvm/llvm-project/pull/122379
More information about the cfe-commits
mailing list