[llvm] RuntimeLibcalls: Add entries for __security_check_cookie (PR #151843)

Matt Arsenault via llvm-commits llvm-commits at lists.llvm.org
Mon Aug 4 06:18:10 PDT 2025


arsenm wrote:

> Is this expected? 

Yes. Literal target checks should be minimized in favor of feature checks 

> Is the presents of security_check_cookie enough to imply isWindowsMSVCEnvironment (not withstanding the current definition of SecurityCheckCookieIfWinMSVC).

I don't know, but does it actually matter? If there's no reason to distinguish the OS when using these calls, it's just duplicating conditions 

> 
> Would is make sense to keep the `if isWindowsMSVCEnvironment` structure but then require/assert the RTLIB::LibcallImpl is supported?

I think that would be worse. I am trying to avoid the current spaghetti of subtlety different correlated conditions used in all of the libcall contexts. 

https://github.com/llvm/llvm-project/pull/151843


More information about the llvm-commits mailing list