For some reason the pre-commit CI of https://reviews.llvm.org/D129233 was all green so I didn't spot this
https://reviews.llvm.org/B174525
Details
Details
Diff Detail
Diff Detail
- Repository
- rG LLVM Github Monorepo
Event Timeline
Comment Actions
LGTM.
libcxx/include/__algorithm/set_intersection.h | ||
---|---|---|
29 | These names look suspicious, the struct has underscores, the members not. Since this patch fixes a CI breakage I prefer to land this patch as is and improve this in a separate patch. For your information __out is a name you can't use. It's used by Microsoft and will fail in the CI in our nasty-macro test. |
Comment Actions
@huixie90 since this fixes the breaking of the ToT I've landed this on your behalf.
Please have a look at the review comment I left earlier.
These names look suspicious, the struct has underscores, the members not. Since this patch fixes a CI breakage I prefer to land this patch as is and improve this in a separate patch.
For your information __out is a name you can't use. It's used by Microsoft and will fail in the CI in our nasty-macro test.