diff options
| author | Mike Beaton <mjsbeaton@gmail.com> | 2024-04-06 23:06:38 +0100 |
|---|---|---|
| committer | mergify[bot] <37929162+mergify[bot]@users.noreply.github.com> | 2024-09-27 16:57:36 +0000 |
| commit | 0aa93aecb7e83b9eeaaac801b1e842eaf6423f08 (patch) | |
| tree | 0289140ca2315525a9ee5a444195df83379f0c27 /MdePkg/Test | |
| parent | cc47e8270375e0254ab7aafadcfde4eec9e01119 (diff) | |
| download | edk2-0aa93aecb7e83b9eeaaac801b1e842eaf6423f08.tar.gz edk2-0aa93aecb7e83b9eeaaac801b1e842eaf6423f08.tar.bz2 edk2-0aa93aecb7e83b9eeaaac801b1e842eaf6423f08.zip | |
MdePkg: Fix DEBUG_CODE and PERF_CODE macros for XCODE5
Without these changes, we get the error:
error: variable '__DebugCodeLocal' set but not used
from the DebugLib.h DEBUG_CODE_BEGIN()/END() macros on XCODE5.
Similarly, in NOOPT builds only, we get:
error: variable '__PerformanceCodeLocal' set but not used
from the PerformanceLib.h PERF_CODE_BEGIN()/END() macros on XCODE5.
It is important to note that the previous code involving a local
variable was intended to ensure correct behaviour of ; following
the macros, in particular that ; should be required:
- https://github.com/tianocore/edk2/pull/6226#issuecomment-2364087866
- https://github.com/tianocore/edk2/pull/6226#issuecomment-2364619759
This converted version repeats the
standard do { ... } while (FALSE) idiom (which is already used in
the END macro) to achieve the same affect.
The modified versions work on all toolchains.
Signed-off-by: Mike Beaton <mjsbeaton@gmail.com>
Diffstat (limited to 'MdePkg/Test')
0 files changed, 0 insertions, 0 deletions