This is an archive of the discontinued LLVM Phabricator instance.

[test] Simplify coro-inline.ll
AbandonedPublic

Authored by aeubanks on Dec 14 2020, 2:59 PM.

Details

Summary

The debug info metadata is unnecessary.
Instead of trying to replicate the -O0 pipeline, just call it.
Use -passes instead of -enable-new-pm.
Remove some unnecessary attributes.

This makes the test pass when -enable-new-pm is on by default.

Diff Detail

Event Timeline

aeubanks created this revision.Dec 14 2020, 2:59 PM
aeubanks requested review of this revision.Dec 14 2020, 2:59 PM
Herald added a project: Restricted Project. · View Herald TranscriptDec 14 2020, 2:59 PM
dongAxis1944 added inline comments.Dec 15 2020, 4:39 AM
llvm/test/Transforms/Coroutines/coro-inline.ll
27

if removing debug info, sample profile might not be found function(@ff) that should be inlined in SampleProfileLoader

aeubanks abandoned this revision.Dec 20 2020, 1:59 PM