tree ab5bc5f0e962fb6fce294a03e10ea728ed77f89d
parent 9057a0026ea9b40744f2eaa91d9540c9bd410079
author Willy Tarreau <w@1wt.eu> 1618075201 +0200
committer Willy Tarreau <w@1wt.eu> 1618075661 +0200
encoding latin1

MINOR: trace: replace the trace() inline function with an equivalent macro

The trace() function is convenient to avoid calling trace() when traces
are not enabled, but there starts to be some callers which place complex
expressions in their trace calls, which results in all of them to be
evaluated before being passed as arguments to the trace() function. This
needlessly wastes precious CPU cycles.

Let's change the function for a macro, so that the arguments are now only
evaluated when the surce has traces enabled. However having a generic
macro being called "trace()" can easily cause conflicts with innocent
code so we rename it "_trace".

Just doing this has resulted in a 2.5% increase of the HTTP/1 request rate.