-
Notifications
You must be signed in to change notification settings - Fork 9
pre-commit: PR151333 #2762
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
pre-commit: PR151333 #2762
Conversation
Diff moderunner: ariselab-64c-docker 8825 files changed, 13051168 insertions(+), 12915993 deletions(-) 6 3 bench/abc/optimized/giaTtopt.ll |
|
The changes primarily involve modifications to
These changes represent a low-level optimization that simplifies pointer arithmetic while maintaining semantic equivalence, potentially improving both compilation efficiency and runtime performance through better optimization opportunities. model: qwen-plus-latest |
| store i8 0, ptr %91, align 1 | ||
| %92 = getelementptr inbounds nuw %struct.aiMetadataEntry, ptr %79, i64 %66, i32 1 | ||
| store ptr %89, ptr %92, align 8 | ||
| store ptr %89, ptr %81, align 8 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Various CSE improvements like this.
| %48 = getelementptr inbounds nuw %"class.std::vector.85", ptr %47, i64 %.015121 | ||
| %49 = load ptr, ptr %48, align 8, !tbaa !287 | ||
| %50 = getelementptr inbounds nuw %"class.std::vector.85", ptr %47, i64 %.015121, i32 0, i32 0, i32 0, i32 1 | ||
| %50 = getelementptr inbounds nuw i8, ptr %48, i64 8 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Also like this, where the common base is CSEd.
| %.idx.i = mul nsw i64 %12, 24 | ||
| %13 = getelementptr i8, ptr %9, i64 %.idx.i | ||
| %14 = getelementptr i8, ptr %13, i64 20 | ||
| %13 = getelementptr %struct.gc_generation, ptr %9, i64 %12 |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
No longer performing ptradd canonicalization in some cases, presumably because https://github.com/llvm/llvm-project/blob/1bd3cc26685636b112d075fd46b20fa0b2e6859f/llvm/lib/Transforms/InstCombine/InstructionCombining.cpp#L3077-L3084 no longer applies. This is ok for now.
|
/add-label reviewed |
Link: llvm/llvm-project#151333
Requested by: @nikic