glaze icon indicating copy to clipboard operation
glaze copied to clipboard

[BUG] MSVC has broken in in several recent versions

Open Kanerudaisuki opened this issue 8 months ago • 6 comments

In order to use the latest version, I've temporarily compiled the code related to the use of glaze using clang-cl alone

Kanerudaisuki avatar May 01 '25 19:05 Kanerudaisuki

Yeah, this is due to MSVC compiler bugs, and made worse by the fact that I cannot run unit tests in the GitHub actions anymore because they introduced new bugs in their compiler (regressed).

Thankfully, I think there is just one active bug left, which you can upvote to encourage them to fix:

https://developercommunity.visualstudio.com/t/Member-pointer-from-base-class-usage-bro/10887486

The latest MSVC fixed their regressed bug that would leak memory and crash the compiler, I'm just waiting until it becomes available in GitHub actions.

stephenberry avatar May 01 '25 20:05 stephenberry

Yeah, this is due to MSVC compiler bugs, and made worse by the fact that I cannot run unit tests in the GitHub actions anymore because they introduced new bugs in their compiler (regressed).是的,这是由于 MSVC 编译器错误造成的,而且更糟糕的是,我无法再在 GitHub 操作中运行单元测试,因为它们在编译器中引入了新的错误(回归)。

Thankfully, I think there is just one active bug left, which you can upvote to encourage them to fix:值得庆幸的是,我认为只剩下一个活跃的错误,您可以投票支持以鼓励他们修复:

https://developercommunity.visualstudio.com/t/Member-pointer-from-base-class-usage-bro/10887486

The latest MSVC fixed their regressed bug that would leak memory and crash the compiler, I'm just waiting until it becomes available in GitHub actions.最新的 MSVC 修复了其回归错误,该错误会泄漏内存并使编译器崩溃,我只是在等待它在 GitHub 操作中可用。

Thank you,The msvc compiler team may not have adequately tested. I've voted and ask others to vote.xD Let's wait for it is avaliable in Github CI

Kanerudaisuki avatar May 02 '25 01:05 Kanerudaisuki

I think I've found an workaround (demo). Do we want to add the workaround to this library?

frederick-vs-ja avatar May 03 '25 04:05 frederick-vs-ja

I think I've found an workaround (demo). Do we want to add the workaround to this library?

MSVC is preparing to release the fix.

Kanerudaisuki avatar May 08 '25 02:05 Kanerudaisuki

Thanks for finding a workaround! But, it does look like a fix has been implemented. So, I'll probably just wait for that.

stephenberry avatar May 08 '25 17:05 stephenberry

#1687 related to this same root cause was fixed by MSVC 17.14 released yesterday (though they did not officially marked it as solved in this released, yet).

dbraeckelmann avatar May 14 '25 08:05 dbraeckelmann