diff options
| author | Andres Freund <andres@anarazel.de> | 2018-11-09 20:43:56 -0800 |
|---|---|---|
| committer | Andres Freund <andres@anarazel.de> | 2018-11-09 20:43:56 -0800 |
| commit | 5fde047f2b41e71706d4ea9e8eb0a401f64192c1 (patch) | |
| tree | bdcd9d4d0a56df9ef9fe246c7cb477c8b947073e /src/backend/access/gist/gistxlog.c | |
| parent | 5fc1670bad9bfa138be1c790d529249e0706d38e (diff) | |
| download | postgresql-5fde047f2b41e71706d4ea9e8eb0a401f64192c1.tar.gz postgresql-5fde047f2b41e71706d4ea9e8eb0a401f64192c1.zip | |
Combine two flag tests in GetSnapshotData().
Previously the code checked PROC_IN_LOGICAL_DECODING and
PROC_IN_VACUUM separately. As the relevant variable is marked as
volatile, the compiler cannot combine the two tests. As
GetSnapshotData() is pretty hot in a number of workloads, it's
worthwhile to fix that.
It'd also be a good idea to get rid of the volatiles altogether. But
for one that's a larger patch, and for another, the code after this
change still seems at least as easy to read as before.
Author: Andres Freund
Discussion: https://postgr.es/m/20181005172955.wyjb4fzcdzqtaxjq@alap3.anarazel.de
Diffstat (limited to 'src/backend/access/gist/gistxlog.c')
0 files changed, 0 insertions, 0 deletions