> > > >1. Why move the assignment to a later place? Can you explain a bit more? > The hang issue here. > > I investigate this issue more. The root cause is that GetProcessAffinityMask() > API requires the 64bit pointer as the input for 64bit ARCH. So, the correct > fix > should declare ProcessAffinityMask and SystemAffinityMask as UINTN. I will > update the patch. > > (GetProcessAffinityMask (GetCurrentProcess (), &ProcessAffinityMask, > &SystemAffinityMask)) >
Thanks! > >2. If CLANG9 is used in Linux, WinHost is still used. That seems not good. > > > Now, CLANG9 is for Windows Emulator only. The key problem is that there is > no way to know which OS is used in DSC. So for now, I will add the limitation > here. For long term, we need to think the way to support Emulator usage. > It's nice to build correct OS-host module depending on compiler tool chain. But if CLANG9 is a cross-host compiler tool chain, I don't think such need should be met. So I would prefer the patch doesn't assume CLANG9 is a hint to build Win host. Which means when using CLANG9, developer needs to use another macro to tell build tool which host to build. > > > >3. Why TftpDynamicCommand cannot be built with CLANG9? > > > CLANG9 supports RC section. I will update patch to remove this change. Thanks. -=-=-=-=-=-=-=-=-=-=-=- Groups.io Links: You receive all messages sent to this group. View/Reply Online (#49152): https://edk2.groups.io/g/devel/message/49152 Mute This Topic: https://groups.io/mt/34540593/21656 Group Owner: devel+ow...@edk2.groups.io Unsubscribe: https://edk2.groups.io/g/devel/unsub [arch...@mail-archive.com] -=-=-=-=-=-=-=-=-=-=-=-
