================ @@ -0,0 +1,100 @@ +.. title:: clang-tidy - bugprone-unsequenced-global-accesses + +bugprone-unsequenced-global-accesses +==================================== + +Finds unsequenced actions (i.e. unsequenced write and read/write) +on global variables nested in functions in the same translation unit. + +Modifying twice or reading and modifying a memory location without a +defined sequence of the operations is either undefined behavior or has +unspecified order. This check is similar to the ``-Wunsequenced`` Clang warning, +however it only looks at global variables and therefore can find unsequenced +actions recursively inside function calls as well. For example: + +.. code-block:: c++ + + int a = 0; + int b = (a++) - a; // This is flagged by -Wunsequenced. + +Because there is no sequencing defined for the ``-`` operator, ``a`` and ``a++`` +could be evaluated in any order. The compiler can even interleave the evaluation +of the sides as this is undefined behavior. The above code would generate a +warning when ``-Wunsequenced`` (or ``-Wsequence-point`` in GCC) is enabled. + +However, global variables allow for more complex scenarios that +``-Wunsequenced`` doesn't detect. E.g. + +.. code-block:: c++ + + int globalVar = 0; + + int incFun() { + globalVar++; + return globalVar; + } + + int main() { + return globalVar + incFun(); // This is not detected by -Wunsequenced. + } + +This clang-tidy check attempts to detect such cases. It recurses into functions ---------------- EugeneZelenko wrote:
```suggestion This check attempts to detect such cases. It recurses into functions ``` https://github.com/llvm/llvm-project/pull/130421 _______________________________________________ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits