tbaederr wrote: `UnsignedOrNone` just means we assert there when passing `-1`. With a full `std::optional<unsigned>` the value would just underflow and the value we use later would've been wrong. The incorrect handling of negative values has nothing to do with `UnsignedOrNone`.
https://github.com/llvm/llvm-project/pull/159579 _______________________________________________ cfe-commits mailing list [email protected] https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits
