This is an archive of the discontinued LLVM Phabricator instance.

[Xtensa] Implement asm representation of the Constant Pool.
Needs RevisionPublic

Authored by andreisfr on Mar 9 2023, 12:16 AM.

Diff Detail

Unit TestsFailed

Event Timeline

andreisfr created this revision.Mar 9 2023, 12:16 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 9 2023, 12:16 AM
Herald added a subscriber: hiraditya. · View Herald Transcript
andreisfr requested review of this revision.Mar 9 2023, 12:16 AM
Herald added a project: Restricted Project. · View Herald TranscriptMar 9 2023, 12:16 AM
andreisfr updated this revision to Diff 508581.Mar 27 2023, 5:00 AM

Minor code formatting

JKRhb added a subscriber: JKRhb.Mar 30 2023, 12:57 PM
arsenm added a comment.Aug 1 2023, 6:05 PM

Tests?

llvm/lib/Target/Xtensa/XtensaAsmPrinter.cpp
106

use a raw_string_ostream with a SmallString?

197–198

Don't need string copy?

arsenm requested changes to this revision.Aug 17 2023, 4:22 PM
This revision now requires changes to proceed.Aug 17 2023, 4:22 PM