Re: [PATCH v3] MdePkg/SynchronizationLib: fix Interlocked[De|In]crement return value

["Kinney, Michael D" <michael.d.kinney@intel.com>]

Laszlo,

Thanks for filing the BZ.  Converting inline assembly to
NASM source files is a really good direction to improve
the readability and maintainability of the code by
consolidating to one assembly source format.

Thanks,
 
Mike

> -----Original Message-----
> From: Laszlo Ersek [mailto:lersek@redhat.com]
> Sent: Tuesday, September 25, 2018 12:26 PM
> To: Ni, Ruiyu <ruiyu.ni@intel.com>; edk2-
> devel@lists.01.org
> Cc: Kinney, Michael D <michael.d.kinney@intel.com>; Yao,
> Jiewen <jiewen.yao@intel.com>; Gao, Liming
> <liming.gao@intel.com>
> Subject: Re: [edk2] [PATCH v3]
> MdePkg/SynchronizationLib: fix Interlocked[De|In]crement
> return value
> 
> On 09/25/18 20:29, Laszlo Ersek wrote:
> 
> > So here's what I'm going to do. I will submit a
> standalone, "surgical"
> > patch, for fixing the regression introduced in
> 17634d026f96.
> 
> Filed
> <https://bugzilla.tianocore.org/show_bug.cgi?id=1207>
> about that.
> 
> > Additionally, I will file a TianoCore BZ about the
> *other* bugs in
> > GccInline.c, and suggest that we fix those issues by
> eliminating the
> > GCC-specific variant altogether.
> 
> That one is
> <https://bugzilla.tianocore.org/show_bug.cgi?id=1208>.
> 
> Thanks
> Laszlo