intel / intel/llvm

Add AdditionalMember to set LangOptions for WorkGroupSize

Open
#3,761 0 comments 0 reactions 1 assignee Claimed by @smanna12 View on GitHub
bug confirmed
Dominant language
LLVM
Stars
1.5k
Forks
854
Avg merge
3d 17h
Merged PRs (30d)
137

Description

I sort of wonder whether we'd be doing a good thing by having an `AdditionalMember` like:
```
const Expr *getWorkGroupSize(const LangOptions &LO) const {
return LO.OpenCL ? getXDim() : getZDim();
}
```
Perhaps not as part of this PR though -- this may be worth thinking about for when we correct the behavior of the one, two, and three arg forms of the attribute.

_Originally posted by @AaronBallman in https://github.com/intel/llvm/pull/3728#discussion_r632719591_

Contributor guide

Open the contributing guide

Assessment

This issue has not been assessed yet.

Get new issues in your inbox

A short digest of beginner-friendly GitHub issues.