Page MenuHomePhabricator

[llvm-exegesis] Set up AsmTargetStreamer in readSnippets
ClosedPublic

Authored by gchatelet on Dec 13 2019, 7:16 AM.

Details

Summary

This is a follow up on D71137 properly setting up the AsmTargetStreamer prior to AsmParser::Run call.

Diff Detail

Event Timeline

gchatelet created this revision.Dec 13 2019, 7:16 AM
mstojanovic accepted this revision.Dec 13 2019, 9:35 AM

LGTM, thanks for the fix!

This revision is now accepted and ready to land.Dec 13 2019, 9:35 AM
courbet added inline comments.Dec 16 2019, 12:58 AM
llvm/tools/llvm-exegesis/lib/SnippetFile.cpp
143

Unless I'm mistaken, this InstrPrinter is leaked here.

gchatelet updated this revision to Diff 234006.Dec 16 2019, 2:43 AM
  • Address comments
This revision was automatically updated to reflect the committed changes.