new lint: warn of dead code when range start is bigger than range end

````rust
fn main()  {
  (4..=1).for_each(|x| println!("{}", x));
}
````
This entire code is dead code and the content of main() is optimized away entirely without any warning.

IMO we should have a warning that suggests `(1..=4).rev()` instead