We often have the following pattern:
static constexpr bool verbose = false;
dataLogLnIf(verbose, "Something is happening");
To make sure that these are always properly eliminated I'd like to make dataLogIf/dataLogLnIf ALWAYS_INLINE.
We may as well mark the branch as UNLIKELY too, for the cases where the condition comes from Options::verboseSomething.
Created attachment 380457 [details]
Comment on attachment 380457 [details]
Clearing flags on attachment: 380457
Committed r250867: <https://trac.webkit.org/changeset/250867>
All reviewed patches have been landed. Closing bug.