diff options
author | Rafael Espindola <rafael.espindola@gmail.com> | 2014-01-22 20:20:52 +0000 |
---|---|---|
committer | Rafael Espindola <rafael.espindola@gmail.com> | 2014-01-22 20:20:52 +0000 |
commit | 28a85a84ac708ce703aa2509ba30ce1dcc453cc3 (patch) | |
tree | 01423cd41118868a20a88771f8ea9594992b401c /llvm/lib/Target/PowerPC/PPCAsmPrinter.cpp | |
parent | 5a8739e0235ee2c5fb09ab2608ac0431698324bb (diff) | |
download | bcm5719-llvm-28a85a84ac708ce703aa2509ba30ce1dcc453cc3.tar.gz bcm5719-llvm-28a85a84ac708ce703aa2509ba30ce1dcc453cc3.zip |
Fix pr18515.
My understanding (from reading just the llvm code) is that
* most ppc cpus have a "sync n" instruction and an msync alias that is "sync 0".
* "book e" cpus instead have a msync instruction and not the more
general "sync n"
This patch reflects that in the .td files, allowing a single codepath for
asm ond obj streamer and incidentelly fixes a crash when EmitRawText was
called on a obj streamer.
llvm-svn: 199832
Diffstat (limited to 'llvm/lib/Target/PowerPC/PPCAsmPrinter.cpp')
-rw-r--r-- | llvm/lib/Target/PowerPC/PPCAsmPrinter.cpp | 7 |
1 files changed, 0 insertions, 7 deletions
diff --git a/llvm/lib/Target/PowerPC/PPCAsmPrinter.cpp b/llvm/lib/Target/PowerPC/PPCAsmPrinter.cpp index 7b0693efce9..15480d5219a 100644 --- a/llvm/lib/Target/PowerPC/PPCAsmPrinter.cpp +++ b/llvm/lib/Target/PowerPC/PPCAsmPrinter.cpp @@ -697,13 +697,6 @@ void PPCAsmPrinter::EmitInstruction(const MachineInstr *MI) { return; } break; - case PPC::SYNC: - // In Book E sync is called msync, handle this special case here... - if (Subtarget.isBookE()) { - OutStreamer.EmitRawText(StringRef("\tmsync")); - return; - } - break; case PPC::LD: case PPC::STD: case PPC::LWA_32: |