{"push_id":2387730681,"size":1,"distinct_size":1,"ref":"refs/heads/master","head":"4d6f85a77157ae42a9dae14a5f5c6357505a8673","before":"786a6065d75247469bb9ba5f50c99bc57a10a31c","commits":[{"sha":"4d6f85a77157ae42a9dae14a5f5c6357505a8673","author":{"name":"Vlad Tsyrklevich","email":"4ac86cc72e249b38f528ae4f2fa3260bc9d23871@tsyrklevich.net"},"message":"[ThinLTO] Keep available_externally symbols live\n\nSummary:\nThis change fixes PR36483. The bug was originally introduced by a change\nthat marked non-prevailing symbols dead. This broke LowerTypeTests\nhandling of available_externally functions, which are non-prevailing.\nLowerTypeTests uses liveness information to avoid emitting thunks for\nunused functions.\n\nMarking available_externally functions dead is incorrect, the functions\nare used though the function definitions are not. This change keeps them\nlive, and lets the EliminateAvailableExternally/GlobalDCE passes remove\nthem later instead.\n\nI've also enabled EliminateAvailableExternally for all optimization\nlevels, I believe it being disabled for O1 was an oversight.\n\nReviewers: pcc, tejohnson\n\nReviewed By: tejohnson\n\nSubscribers: grimar, mehdi_amini, inglorion, eraman, llvm-commits\n\nDifferential Revision: https://reviews.llvm.org/D43690\n\nllvm-svn=327041","distinct":true,"url":"https://api.github.com/repos/jyknight/llvm-monorepo/commits/4d6f85a77157ae42a9dae14a5f5c6357505a8673"}]}
{"actor":{"display_login":"jyknight"}}