This is an archive of the discontinued LLVM Phabricator instance.

[FuncSpec][NFC] Improve the unittest coverage for constant folding of GEPs.
ClosedPublic

Authored by labrinea on Jul 10 2023, 1:42 AM.

Details

Summary

The InstCostVisitor is currently using the DataLayout-independent constant folding interface.
This is a workaround since we can't directly call ConstantExpr::getGetElementPtr due to deprecation.
This patch shows that the constant folding interface we are using is not good enough.

Diff Detail

Event Timeline

labrinea created this revision.Jul 10 2023, 1:42 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 10 2023, 1:42 AM
Herald added a subscriber: ormris. · View Herald Transcript
labrinea requested review of this revision.Jul 10 2023, 1:42 AM
Herald added a project: Restricted Project. · View Herald TranscriptJul 10 2023, 1:42 AM
labrinea edited the summary of this revision. (Show Details)Jul 10 2023, 7:05 AM
ChuanqiXu accepted this revision.Jul 10 2023, 6:53 PM
This revision is now accepted and ready to land.Jul 10 2023, 6:53 PM