So a big part of the problem here is that the question is asking a whole bunch of different questions. (4, specifically). Of those, two are asking for opinions, and two are not. It's worth noting that in the comments and answers, everyone has basically just ignored the two questions asking for opinions, and pretty much just pretended that the question was asking the two objective questions that it asked (which is really just the same question phrased twice, making it really just one question).
To be more specific, the first question is:
Should I prefer handling errors with if unlikely?
Which is indeed just straight up asking for people's opinions.
The second question however is:
Will the compiler automatically deduce which part of the code should be cached
Which is an objective question with a verifiably correct answer. It's also the answer that the answers focused on, discussing the effect that the change in question has on the compiled code (and whether or not there is one).
Answers did go on, after answering that question, to go over other changes, besides just the one mentioned in the question, that would also have the same effect, in order to demonstrate different ways that someone can accomplish the same effect the question is indicating is desirable. This is appropriate. Adding additional relevant, but tangential, information to an answer after answering the question asked is appropriate.
The existence of the second, objective, question also adds context to the first question. Looking at the first question in a vacuum, it's simply asking for opinions, but given that the question goes on to discuss whether or not the stated change affects the ability of the CPU to properly cache information, it has added context to the question indicating that the OP wants to ensure that the code is cached properly, and wants to know which solutions accomplish that goal based on their preferences, rather than asking other people whether they would prefer to cache it or not.
So, given all of this, we've ended up in a position where a question technically asks an opinion based question, but everyone is ignoring that entirely and just answering the objective question that it also asked, so the solution here isn't to close the question as opinion based, but rather to simply edit out the question asking for opinions, and to leave the objective question that everyone was treating as the only question all along. I've since applied this edit.
if unlikely(...)
does means that it's really hard to evaluate if the answer is actually answering the question (especially when the actual, technical question is buried and is lacking a?
). If I'd seen it as a review I'd have opened the new tab, read things over and still been at a loss as to whether or not this was a good question.