microsoft / microsoft/DirectXShaderCompiler
Free operator overload
Open
Nobody has claimed this yet.
hlsl-next
- Dominant language
- C++
- Stars
- 3.7k
- Forks
- 900
- Avg merge
- 2d 11h
- Merged PRs (30d)
- 44
Description
If you want to allow "class + x" and "x + class" you need to have free operator overloading, is this possible or is there another way around it? If not, could this be a future feature?
template<typename T, uint32_t N>
class array
{
T mArr[N];
void set(const T value) {
[unroll] for(uint i = 0 ; i < N; i++) { mArr[i] = value; }
}
float operator[](const uint32_t pos)
{ return mArr[pos]; }
};
template<typename T, uint32_t N>
array<T,N> operator+ (array<T,N> lhs, T rhs) {
array<T,N> arr;
[unroll] for(uint i = 0 ; i < N; i++) { arr.mArr[i] = lhs.mArr[i] + rhs; }
return arr;
}
template<typename T, uint32_t N>
array<T,N> operator+ (T lhs, array<T,N> rhs) {
array<T,N> arr;
[unroll] for(uint i = 0 ; i < N; i++) { arr.mArr[i] = lhs + rhs.mArr[i]; }
return arr;
}
[numthreads(1, 1, 1)]
void main( uint3 DTid : SV_DispatchThreadID )
{
array<float, 3> arr1;
array<float, 3> arr2;
arr1.set(2.0);
arr2.set(2.0);
array<float, 3> result = arr1 + 2.0f;
}
Contributor guide
First steps
- Read the whole issue, then the project's contributing guide.
- Comment on the issue to say you are picking it up — it saves two people doing the same work.
- Fork the repository and make your change on a branch.
- Open a pull request that references the issue number.
Research direction
No source files, tests, or entry points are named. Start by locating the compiler's handling of operator overloading and checking whether free operators are currently supported for the shown array pattern. Done would require a defined implementation scope, corresponding coverage, and a maintainer decision on whether the feature belongs in the language.
Written by the indexing model from the issue text.
Assessment
- Tech stack
- cpp
- Domain
- compilers
- Issue type
- Feature
- Difficulty
- 5/5
- Estimated time
- Over a week
- Activity status
- Stale
- Clarity
- Needs clarification
- Newbie friendliness
- 20/100