diff options
author | Louis Dionne <ldionne.2@gmail.com> | 2021-12-08 08:26:27 -0500 |
---|---|---|
committer | Louis Dionne <ldionne.2@gmail.com> | 2021-12-13 14:37:54 -0500 |
commit | 515afe8b13a659a7a979280ed87891e74cc7915c (patch) | |
tree | 0b22195bb4b776be7a2d040454cca4d2d48fd21d /lldb/unittests/ScriptInterpreter/Python/PythonTestSuite.cpp | |
parent | 206365bf8fdf6425cba89b50d120a6a9f82a0459 (diff) | |
download | llvm-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