[PATCH] Don't report the error when non-dangerous load/store is in branch delay slot

Mark Seaborn mseaborn at chromium.org
Fri Jun 6 15:30:03 PDT 2014


================
Comment at: lib/Target/Mips/MCTargetDesc/MipsNaClELFStreamer.cpp:151
@@ +150,3 @@
+
+    // Sandbox loads, stores and SP changes.
+    unsigned AddrIdx;
----------------
I don't get why you've reordered this chunk of code.  Surely the ordering doesn't matter, because Inst can't be both a call and a memory access on MIPS, can it?

http://reviews.llvm.org/D4048






More information about the llvm-commits mailing list