diff options
author | Laszlo Ersek <lersek@redhat.com> | 2018-09-29 21:22:57 +0200 |
---|---|---|
committer | Laszlo Ersek <lersek@redhat.com> | 2018-10-17 18:44:48 +0200 |
commit | e5d4e7500fc92475d079d16846671ecbbb08e8af (patch) | |
tree | b18ae90ed7879b900ee4df30066ee994b89f3c70 /BaseTools/Source/Python/Workspace | |
parent | 7149d40938f85cfe4c29a57c7c1126247ab55b49 (diff) | |
download | edk2-e5d4e7500fc92475d079d16846671ecbbb08e8af.tar.gz edk2-e5d4e7500fc92475d079d16846671ecbbb08e8af.tar.bz2 edk2-e5d4e7500fc92475d079d16846671ecbbb08e8af.zip |
MdePkg/BaseSynchronizationLib GCC: fix X64 InternalSyncCompareExchange64()
(This patch is identical to the X64 half of the last one, except for the
InternalSyncCompareExchange32() -> InternalSyncCompareExchange64() and
"cmpxchgl" -> "cmpxchgq" replacements.)
The CMPXCHG instruction has the following operands:
- AX (implicit, CompareValue): input and output
- destination operand (*Value): input and output
- source operand (ExchangeValue): input
The X64 version of InternalSyncCompareExchange64() attempts to mark both
CompareValue and (*Value) as input/output, but it doesn't use the
appropriate constraints for either operand.
Fix these issues. Furthermore, prefer the short "+" constraint for I/O
operands over the <output-operand-number> constraint that can be applied
to the input instances of I/O operands.
Cc: Liming Gao <liming.gao@intel.com>
Cc: Michael D Kinney <michael.d.kinney@intel.com>
Ref: https://bugzilla.tianocore.org/show_bug.cgi?id=1208
Contributed-under: TianoCore Contribution Agreement 1.1
Signed-off-by: Laszlo Ersek <lersek@redhat.com>
Reviewed-by: Philippe Mathieu-Daudé <philmd@redhat.com>
Acked-by: Michael D Kinney <michael.d.kinney@intel.com>
Reviewed-by: Liming Gao <liming.gao@intel.com>
Diffstat (limited to 'BaseTools/Source/Python/Workspace')
0 files changed, 0 insertions, 0 deletions