Add AdditionalMember to set LangOptions for WorkGroupSize
Open
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
Assessment
This issue has not been assessed yet.