Details
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Thank you so much for doing this. I'm hopeful our hackery can go away sometime soon.
clang/test/CXX/drs/dr25xx.cpp | ||
---|---|---|
3 | If you could add a comment explaining this status for me, I would be grateful: // We've implemented DR2565 as proposed (that is, a substitution failure in a requires parameter list causes the requires clause to evaluate as false) as it seems to make the most sense, and fit best into our implementation. If this issue gets accepted as-is, this can be marked as complete. | |
clang/www/make_cxx_dr_status | ||
161–164 | I guess this is OK, its a shame we have to custom-tag these, but I guess I'm 1/2 responsible here :/ |
clang/test/CXX/drs/dr25xx.cpp | ||
---|---|---|
3 | You said we were in a position where we had 1/2 of it implemented. So I put it as partial. I must've misunderstood you. Will change it back. | |
clang/www/make_cxx_dr_status | ||
161–164 | Yeah, it looks hacky. If the list gets larger, we could do something like namespace dr1432 { // dr1432: open 16 to make it more general. |
If you could add a comment explaining this status for me, I would be grateful:
// We've implemented DR2565 as proposed (that is, a substitution failure in a requires parameter list causes the requires clause to evaluate as false) as it seems to make the most sense, and fit best into our implementation. If this issue gets accepted as-is, this can be marked as complete.