-
Notifications
You must be signed in to change notification settings - Fork 10.5k
[SILOpt] Disabled DestroyHoisting. #41478
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
[SILOpt] Disabled DestroyHoisting. #41478
Conversation
|
|
e48a4ac
to
777dcb5
Compare
|
@swift-ci please test |
54867dd
to
6057e37
Compare
|
6057e37
to
72c6b2c
Compare
@swift-ci please benchmark |
1 similar comment
@swift-ci please benchmark |
|
33d268d
to
70d6471
Compare
@swift-ci please benchmark |
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.
Great!
- referring the new "fixed pointer handling commit"
@swift-ci please test |
@swift-ci please benchmark |
@swift-ci please test windows platform |
lib/SIL/Utils/MemAccessUtils.cpp
Outdated
void swift::visitProductLeafAccessPathNodes(SILValue address, | ||
TypeExpansionContext tec, | ||
SILModule &module, | ||
std::function<void(AccessPath::PathNode, SILType)> visitor) { |
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.
swift::visitProductLeafAccessPathNodes
Very nice!
return true; | ||
} else { | ||
assert(load->getOperand()->getType().isTrivial(*load->getFunction())); | ||
bool HoistDestroys::foldBarrier(SILInstruction *barrier, |
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.
foldBarrier
This is awesome. I'm thrilled that it was so easy to do now.
8b0d852
to
8626cae
Compare
@swift-ci please benchmark |
@swift-ci please test windows |
@swift-ci please benchmark |
8626cae
to
d094b19
Compare
@swift-ci please test source compatibility |
@swift-ci please test macos platform |
@swift-ci please benchmark |
211ca19
to
5b56b76
Compare
|
6cc9875
to
017e8b8
Compare
@swift-ci please benchmark |
017e8b8
to
12a15bd
Compare
@swift-ci please benchmark |
12a15bd
to
672ec76
Compare
@swift-ci please test |
@swift-ci please test source compatibility |
1 similar comment
@swift-ci please test source compatibility |
@swift-ci please test |
@swift-ci please test source compatibility |
@swift-ci please test source compatibility release |
f04c9a3
to
979196c
Compare
@swift-ci please test |
@swift-ci please test linux platform |
@swift-ci please clean test linux platform |
2 similar comments
@swift-ci please clean test linux platform |
@swift-ci please clean test linux platform |
@swift-ci please test linux platform |
No description provided.