convert switch id check with architecture to warning instead of error

[saaramahmoudi]

This PR converts the switch error check with the architecture to a warning.

@vaughnbetz

[vaughnbetz]

Can you summarize the testing for this case? The nightly tests aren't running automatically right now, so if you're not sure if this case is tested in the github runners you should run them manually.

Do we need any new tests?

[vaughnbetz]

Probably we should not make this a warning, and instead just make it an info message saying something like:
"Switch name '%s' found in RR graph input from file but not in architecture file; creating it.\n", string_name.c_str());

Otherwise you are likely to get many warnings with the RR graphs you are bringing in, and they will be spurious.

[saaramahmoudi]

@vaughnbetz I don't think there is a need to add new tests. However, I have ran the titan benchmarks locally and made sure that nothing is going to break with this change.
We also treat internal switches for flat router with the same logic, we don't look for them in the architecture file and we just don't print any error with those. So, I think it should be working fine

[vaughnbetz]

Thanks. One nit: there should be a semicolon (or a period and new sentence if you prefer) before "creating it." i.e. "; creating it."

[saaramahmoudi]

@vaughnbetz resolved