CARVIEW |
Select Language
HTTP/2 200
date: Sat, 11 Oct 2025 15:17:27 GMT
content-type: text/html; charset=utf-8
cache-control: max-age=0, private, must-revalidate
content-security-policy: default-src 'none'; base-uri 'self'; child-src github.githubassets.com github.com/assets-cdn/worker/ github.com/assets/ gist.github.com/assets-cdn/worker/; connect-src 'self' uploads.github.com www.githubstatus.com collector.github.com raw.githubusercontent.com api.github.com github-cloud.s3.amazonaws.com github-production-repository-file-5c1aeb.s3.amazonaws.com github-production-upload-manifest-file-7fdce7.s3.amazonaws.com github-production-user-asset-6210df.s3.amazonaws.com *.rel.tunnels.api.visualstudio.com wss://*.rel.tunnels.api.visualstudio.com github.githubassets.com objects-origin.githubusercontent.com copilot-proxy.githubusercontent.com proxy.individual.githubcopilot.com proxy.business.githubcopilot.com proxy.enterprise.githubcopilot.com *.actions.githubusercontent.com wss://*.actions.githubusercontent.com productionresultssa0.blob.core.windows.net/ productionresultssa1.blob.core.windows.net/ productionresultssa2.blob.core.windows.net/ productionresultssa3.blob.core.windows.net/ productionresultssa4.blob.core.windows.net/ productionresultssa5.blob.core.windows.net/ productionresultssa6.blob.core.windows.net/ productionresultssa7.blob.core.windows.net/ productionresultssa8.blob.core.windows.net/ productionresultssa9.blob.core.windows.net/ productionresultssa10.blob.core.windows.net/ productionresultssa11.blob.core.windows.net/ productionresultssa12.blob.core.windows.net/ productionresultssa13.blob.core.windows.net/ productionresultssa14.blob.core.windows.net/ productionresultssa15.blob.core.windows.net/ productionresultssa16.blob.core.windows.net/ productionresultssa17.blob.core.windows.net/ productionresultssa18.blob.core.windows.net/ productionresultssa19.blob.core.windows.net/ github-production-repository-image-32fea6.s3.amazonaws.com github-production-release-asset-2e65be.s3.amazonaws.com insights.github.com wss://alive.github.com wss://alive-staging.github.com api.githubcopilot.com api.individual.githubcopilot.com api.business.githubcopilot.com api.enterprise.githubcopilot.com; font-src github.githubassets.com; form-action 'self' github.com gist.github.com copilot-workspace.githubnext.com objects-origin.githubusercontent.com; frame-ancestors 'none'; frame-src viewscreen.githubusercontent.com notebooks.githubusercontent.com; img-src 'self' data: blob: github.githubassets.com media.githubusercontent.com camo.githubusercontent.com identicons.github.com avatars.githubusercontent.com private-avatars.githubusercontent.com github-cloud.s3.amazonaws.com objects.githubusercontent.com release-assets.githubusercontent.com secured-user-images.githubusercontent.com/ user-images.githubusercontent.com/ private-user-images.githubusercontent.com opengraph.githubassets.com marketplace-screenshots.githubusercontent.com/ copilotprodattachments.blob.core.windows.net/github-production-copilot-attachments/ github-production-user-asset-6210df.s3.amazonaws.com customer-stories-feed.github.com spotlights-feed.github.com objects-origin.githubusercontent.com *.githubusercontent.com; manifest-src 'self'; media-src github.com user-images.githubusercontent.com/ secured-user-images.githubusercontent.com/ private-user-images.githubusercontent.com github-production-user-asset-6210df.s3.amazonaws.com gist.github.com; script-src github.githubassets.com; style-src 'unsafe-inline' github.githubassets.com; upgrade-insecure-requests; worker-src github.githubassets.com github.com/assets-cdn/worker/ github.com/assets/ gist.github.com/assets-cdn/worker/
link: ; rel=preload; as=fetch; crossorigin=use-credentials
referrer-policy: no-referrer-when-downgrade
server-timing: issue_layout-fragment;desc="issue_layout fragment";dur=263.440792,issue_conversation_content-fragment;desc="issue_conversation_content fragment";dur=754.117275,nginx;desc="NGINX";dur=1.220753,glb;desc="GLB";dur=137.420693
strict-transport-security: max-age=31536000; includeSubdomains; preload
vary: X-PJAX, X-PJAX-Container, Turbo-Visit, Turbo-Frame, X-Requested-With, Accept,Accept-Encoding, Accept, X-Requested-With
x-content-type-options: nosniff
x-frame-options: deny
x-voltron-version: aab62e3
x-xss-protection: 0
server: github.com
content-encoding: gzip
accept-ranges: bytes
set-cookie: _gh_sess=28DPB9tVbjdvqcuT5mP4Ys6dPHoLDYfU%2FmuIExuLKRioIiwk5PP3dJHc63oJOK%2BgSP8kalMtJMCrX5cIX1aNew4UqPB47rFoj7Ie9QT5QuXrthfuQxGBtI6oPeojO694dmm5JWC8uBDSZrZVwzly7EurAJ8Le8iiMMbzd4WozkFjddGd3C5koa0NssDiE9b50wmfPlwH92WvNfyHoTuGovTemv%2FrNWemyW3MgI6c0zp9wLFIN2y5skSqhAWcC5hdngh8uEqLFHgEeRMbGcH3iA%3D%3D--zL5Jh%2BgLT4SRpHBQ--rcDO%2Bg%2FNwRIeuF41k%2Fbf%2Fw%3D%3D; Path=/; HttpOnly; Secure; SameSite=Lax
set-cookie: _octo=GH1.1.287487789.1760195846; Path=/; Domain=github.com; Expires=Sun, 11 Oct 2026 15:17:26 GMT; Secure; SameSite=Lax
set-cookie: logged_in=no; Path=/; Domain=github.com; Expires=Sun, 11 Oct 2026 15:17:26 GMT; HttpOnly; Secure; SameSite=Lax
x-github-request-id: A6AA:2994BB:7DBA55:A5D355:68EA7506
missed optimization for ceiling division with known ranges · Issue #142497 · llvm/llvm-project · GitHub
No one assignedNo typeNo projectsNo milestoneNone yetNo branches or pull requests
Skip to content
Navigation Menu
{{ message }}
-
Notifications
You must be signed in to change notification settings - Fork 15.2k
Closed
Labels
llvm:instcombineCovers the InstCombine, InstSimplify and AggressiveInstCombine passesCovers the InstCombine, InstSimplify and AggressiveInstCombine passesmissed-optimization
Description
My starting point was the following Rust code (meaning is hopefully clear even to non-rust speakers):
pub fn src(v: u32) -> u32 {
(u32::BITS - v.leading_zeros()).div_ceil(8)
}
rustc emits the following LLVM IR:
define noundef range(i32 0, 6) i32 @src(i32 noundef %v) unnamed_addr #0 {
start:
%0 = tail call range(i32 0, 33) i32 @llvm.ctlz.i32(i32 %v, i1 false)
%_2 = sub nuw nsw i32 32, %0
%_41 = lshr i32 %_2, 3
%_5 = and i32 %_2, 7
%_6.not = icmp ne i32 %_5, 0
%1 = zext i1 %_6.not to i32
%_0.sroa.0.0 = add nuw nsw i32 %_41, %1
ret i32 %_0.sroa.0.0
}
Which emits the following x86:
src: # @src
mov ecx, 63
bsr ecx, edi
xor ecx, -32
add ecx, 33
mov eax, ecx
shr eax, 3
and ecx, 7
cmp ecx, 1
sbb eax, -1
ret
however, this could be validly optimized to the following LLVM-IR:
define noundef range(i32 0, 5) i32 @tgt(i32 noundef %v) unnamed_addr #0 {
start:
%0 = tail call i32 @llvm.ctlz.i32(i32 %v, i1 false)
%1 = sub nuw nsw i32 32, %0
%2 = add nuw nsw i32 %1, 7
%3 = lshr i32 %2, 3
ret i32 %3
}
which produces the following, much tighter x86:
tgt: # @tgt
mov eax, 63
bsr eax, edi
xor eax, -32
add eax, 40
shr eax, 3
ret
alive2 showing that the transformation is valid: https://alive2.llvm.org/ce/z/Ys4qAy
(As a bit of interest, I found the optimized versions using claude. Computers are wild: https://claude.ai/share/d998511d-45ee-4132-bee4-fe7f70350a67)
Metadata
Metadata
Assignees
Labels
llvm:instcombineCovers the InstCombine, InstSimplify and AggressiveInstCombine passesCovers the InstCombine, InstSimplify and AggressiveInstCombine passesmissed-optimization
Type
Projects
Milestone
Relationships
Development
Issue actions
You can’t perform that action at this time.