Skip to content

Conversation

@v-klochkov
Copy link
Contributor

This patch fixes 2 constructors, fixes get_count() and get_size() methods,
adds operator== and operator!=, fixes hash implementation,
adds static_asssert to range and id classes to have early check for wrong
dimension, fixes few other errors.

Signed-off-by: Vyacheslav N Klochkov vyacheslav.n.klochkov@intel.com

@v-klochkov v-klochkov requested review from bader and vladimirlaz May 23, 2019 17:19
This patch fixes 2 constructors, fixes get_count() and get_size() methods,
adds operator== and operator!=, fixes hash implementation,
adds static_asssert to range and id classes to have early check for wrong
dimension, fixes few other errors.

Signed-off-by: Vyacheslav N Klochkov <vyacheslav.n.klochkov@intel.com>
@v-klochkov v-klochkov force-pushed the public_acc20_fixes branch from e7ca387 to 14b635a Compare May 24, 2019 16:09
Copy link
Contributor

@bader bader left a comment

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Thanks!
Very good improvement.

@bader bader merged commit a91d722 into intel:sycl May 26, 2019
@v-klochkov v-klochkov deleted the public_acc20_fixes branch May 28, 2019 15:08
vmaksimo pushed a commit to vmaksimo/llvm that referenced this pull request Jan 18, 2021
  CONFLICT (content): Merge conflict in clang/lib/Frontend/CompilerInvocation.cpp
  CONFLICT (content): Merge conflict in clang/include/clang/Driver/Options.td
coldav pushed a commit to coldav/llvm that referenced this pull request Aug 22, 2025
[vecz] Don't mask work-group collective operations
coldav pushed a commit to coldav/llvm that referenced this pull request Aug 26, 2025
[vecz] Don't mask work-group collective operations
coldav pushed a commit to coldav/llvm that referenced this pull request Aug 29, 2025
[vecz] Don't mask work-group collective operations
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

2 participants