aboutsummaryrefslogtreecommitdiff
path: root/lldb/unittests/ScriptInterpreter/Python/PythonTestSuite.cpp
diff options
context:
space:
mode:
authorLouis Dionne <ldionne.2@gmail.com>2021-12-08 08:26:27 -0500
committerLouis Dionne <ldionne.2@gmail.com>2021-12-13 14:37:54 -0500
commit515afe8b13a659a7a979280ed87891e74cc7915c (patch)
tree0b22195bb4b776be7a2d040454cca4d2d48fd21d /lldb/unittests/ScriptInterpreter/Python/PythonTestSuite.cpp
parent206365bf8fdf6425cba89b50d120a6a9f82a0459 (diff)
downloadllvm-515afe8b13a659a7a979280ed87891e74cc7915c.zip
llvm-515afe8b13a659a7a979280ed87891e74cc7915c.tar.gz
llvm-515afe8b13a659a7a979280ed87891e74cc7915c.tar.bz2
[libc++] Change workaround for init_priority((100)) outside of system headers
We had previously been using a different workaround for pretending that we were inside a system header, however it had some undesirable effects on dependency parsing for build systems, as explained in [1]. This patch changes the workaround to use `#pragma GCC system_header`, which shouldn't suffer from the same issue. Unfortunately, it is a lot more verbose. The issue is that `#pragma GCC system_header` is ignored when we are inside a source file, so we have to create a header just for the sake of using it. IMO this seems like an artificial restriction without much benefit, but investigating that is a different story. For now, this should at least solve build system problems at the cost of some readability. [1]: https://reviews.llvm.org/D95972#3178968 Differential Revision: https://reviews.llvm.org/D115334
Diffstat (limited to 'lldb/unittests/ScriptInterpreter/Python/PythonTestSuite.cpp')
0 files changed, 0 insertions, 0 deletions