<html xmlns:v="urn:schemas-microsoft-com:vml" xmlns:o="urn:schemas-microsoft-com:office:office" xmlns:w="urn:schemas-microsoft-com:office:word" xmlns:m="http://schemas.microsoft.com/office/2004/12/omml" xmlns="http://www.w3.org/TR/REC-html40"> <head> <meta http-equiv="Content-Type" content="text/html; charset=utf-8"> <meta name="Generator" content="Microsoft Word 15 (filtered medium)"> <style></style> </head> <body lang="EN-US" link="blue" vlink="purple"> <div class="WordSection1"> <p class="MsoNormal">Vitaly:<o:p></o:p></p> <p class="MsoNormal"> Yes. I think we should have better solution in OpenSSL to support EDK2 usage. But, this is a long term solution. For now, I will try the solution to remove -fms-compatibility option in CLANGPDB tool chain. <o:p></o:p></p> <p class="MsoNormal"><o:p> </o:p></p> <p class="MsoNormal">Thanks<o:p></o:p></p> <p class="MsoNormal">Liming<o:p></o:p></p> <div style="border:none;border-left:solid blue 1.5pt;padding:0in 0in 0in 4.0pt"> <div> <div style="border:none;border-top:solid #E1E1E1 1.0pt;padding:3.0pt 0in 0in 0in"> <p class="MsoNormal"><b>From:</b> vit9696 <vit9696@protonmail.com> <br> <b>Sent:</b> Monday, February 3, 2020 7:29 PM<br> <b>To:</b> Gao, Liming <liming.gao@intel.com>; devel@edk2.groups.io; Marvin Häuser <marvin.haeuser@outlook.com><br> <b>Subject:</b> RE: [edk2-devel] [PATCH 0/1] Use _MSC_VER to determine MSVC compiler<o:p></o:p></p> </div> </div> <p class="MsoNormal"><o:p> </o:p></p> <div> <p class="MsoNormal">Liming,<o:p></o:p></p> </div> <div> <p class="MsoNormal"><o:p> </o:p></p> </div> <p class="MsoNormal">I believe it is reasonable to fix OpenSSL, but most likely it is faster to address EDK II code at first. In future we should still stick to _MSC_VER, but for now just ensuring that any toolchain by MSVC does not define _MSC_EXTENSIONS should work too.<o:p></o:p></p> <div> <p class="MsoNormal"><o:p> </o:p></p> </div> <div> <p class="MsoNormal">I believe that -fms-compatibility option is not needed for CLANGPDB, as CLANGPDB is literally using clang, and that worked fine before in CLANG38 and XCODE5. We may still have to update some preprocessor conditions to include __clang__ near __GNUC__ if __GNUC__ is left undefined even when we remove -fms-compatibility, but that sounds fine for me.<o:p></o:p></p> </div> <div> <p class="MsoNormal"><o:p> </o:p></p> </div> <div> <p class="MsoNormal">We will investigate that on our own and submit a new patch, but help from other parties is strongly appreciated. We mostly use XCODE5 and are not well aware of other platforms.<o:p></o:p></p> </div> <div> <p class="MsoNormal"><o:p> </o:p></p> </div> <div> <p class="MsoNormal">Best wishes,<o:p></o:p></p> </div> <div> <p class="MsoNormal">Vitaly<o:p></o:p></p> </div> <div> <div> <div> <p class="MsoNormal"><o:p> </o:p></p> </div> <p class="MsoNormal">On Mon, Feb 3, 2020 at 11:00, Gao, Liming <<a href="mailto:liming.gao@intel.com">liming.gao@intel.com</a>> wrote:<o:p></o:p></p> <blockquote style="margin-top:5.0pt;margin-bottom:5.0pt"> <p class="MsoNormal" style="margin-bottom:12.0pt">Vitaly:<br> This change will cause CryptoPkg openssl build failure, because OpensslLib.inf undefines _MSC_VER macro to make sure openssl source code be built in edk2 project without using MS VS functions.<br> <br> Now, I have no good solution to fix this issue. One way is to use defined(_MSC_EXTENSIONS) && !defined(__clang__), another way is to investigate whether we can remove -fms-compatibility option in CLANGPDB.<br> <br> Thanks<br> Liming<br> > -----Original Message-----<br> > From: <a href="mailto:devel@edk2.groups.io">devel@edk2.groups.io</a> <<a href="mailto:devel@edk2.groups.io">devel@edk2.groups.io</a>> On Behalf Of Vitaly Cheptsov via Groups.Io<br> > Sent: Saturday, February 1, 2020 5:17 AM<br> > To: <a href="mailto:devel@edk2.groups.io">devel@edk2.groups.io</a><br> > Subject: [edk2-devel] [PATCH 0/1] Use _MSC_VER to determine MSVC compiler<br> ><br> > Patch details are explained in <a href="https://bugzilla.tianocore.org/show_bug.cgi?id=2397"> https://bugzilla.tianocore.org/show_bug.cgi?id=2397</a>.<br> > We request this to be merged in edk2-stable202002.<br> ><br> > Vitaly Cheptsov (1):<br> > MdePkg: Use _MSC_VER to determine MSVC compiler<br> ><br> > MdePkg/Include/AArch64/ProcessorBind.h | 2 +-<br> > MdePkg/Include/Arm/ProcessorBind.h | 8 ++++----<br> > MdePkg/Include/Base.h | 10 +++++-----<br> > MdePkg/Include/Ia32/ProcessorBind.h | 6 +++---<br> > MdePkg/Include/X64/ProcessorBind.h | 6 +++---<br> > 5 files changed, 16 insertions(+), 16 deletions(-)<br> ><br> > --<br> > 2.21.1 (Apple Git-122.3)<br> ><br> ><br> > -=-=-=-=-=-=<br> > Groups.io Links: You receive all messages sent to this group.<br> ><br> > View/Reply Online (#53623): <a href="https://edk2.groups.io/g/devel/message/53623"> https://edk2.groups.io/g/devel/message/53623</a><br> > Mute This Topic: <a href="https://groups.io/mt/70882954/1759384">https://groups.io/mt/70882954/1759384</a><br> > Group Owner: <a href="mailto:devel+owner@edk2.groups.io">devel+owner@edk2.groups.io</a><br> > Unsubscribe: <a href="https://edk2.groups.io/g/devel/unsub">https://edk2.groups.io/g/devel/unsub</a> [liming.gao@intel.com]<br> > -=-=-=-=-=-=<o:p></o:p></p> </blockquote> <div> <p class="MsoNormal"><o:p> </o:p></p> </div> <div> <p class="MsoNormal"><o:p> </o:p></p> </div> </div> </div> </div> </div> </body> </html> <div width="1" style="color:white;clear:both">_._,_._,_</div> <hr> Groups.io Links:<p> You receive all messages sent to this group. <p> <a target="_blank" href="https://edk2.groups.io/g/devel/message/53700">View/Reply Online (#53700)</a> | | <a target="_blank" href="https://groups.io/mt/70882954/1813853">Mute This Topic</a> | <a href="https://edk2.groups.io/g/devel/post">New Topic</a><br> <br> <a href="https://edk2.groups.io/g/devel/editsub/1813853">Your Subscription</a> | <a href="mailto:devel+owner@edk2.groups.io">Contact Group Owner</a> | <a href="https://edk2.groups.io/g/devel/unsub">Unsubscribe</a> [edk2-devel-archive@redhat.com]<br> <div width="1" style="color:white;clear:both">_._,_._,_</div>