Pattern matching regression on numbers (at least)

[painedpineapple]

When switching on numbers (like HTTP status codes), the compiled output is incorrect.

Example:

let httpResponseCode = 201
let _ =
  match httpResponseCode with
  | 201|200 ->
      Js.log (("good response")[@reason.raw_literal "good response"])
  | 500|503|506|509|598|501|504|507|510|599|502|505|508|511 ->
      Js.log (("bad response")[@reason.raw_literal "bad response"])
  | _ -> Js.log (("the catch all")[@reason.raw_literal "the catch all"])

Reason playground: https://reasonml.github.io/en/try?rrjsx=true&reason=DYUwLgBAFmYA4CUQGc4HsB2yQGE0BMQIBeCAJgAYBGAbgCg7kB3ASzAGMoIAKGeJVJmx5CASggBvOgB9y1GXIokAfHQgQAUsgB0wNAHNuAIn1oCEAE4p0WEEdH1ZAVgoUFLgMzuKANm8BOd38ADm8qbwAWbwB2dyo3Z39A5woybydvUOcqKhU1TR09QyMAIwBDfEtrITtRBQB9PPUtXQNjMCgidjKOLjLgYHt6AF8aIA

In the example, if you remove all of the 5XX numbers or remove only 502 it'll work correctly. Other than that whenever the status code is 201 or 200 at least it hits the catch-all instead of the correct case.

[anmonteiro]

Is this also related to #3875?

[mlms13]

I think #3875 is a regression with 5.2.0, but from the comments on Discord, it seems like the issue here is also present in 5.0.3, 5.0.4, and 5.0.6.

[bobzhang]

@thislogancall This is an old embarrassing bug, fixed in #3881