https://rise4fun.com/Alive/IT3
Comes up in the [most ugliest] signed int -> signed char case of
-fsanitize=implicit-conversion (https://reviews.llvm.org/D50250)
Previously, we were stuck with not:
But now we are able to completely get rid of it:
(FIXME: why are we loosing the metadata? that seems wrong/strange.)
Not sure if we want to do it always, or only when it is free to invert?
The one-use here is not strictly necessary? There are questions about how to best deal with inverted compares, so I don't think it needs to hold this patch up, but it's worth a code comment if I'm seeing it correctly.
This reminded me of the still open:
D35182