Subzero: Add Non-SFI support for x86-32.

src/IceInst.cpp

Index: src/IceInst.cpp
diff --git a/src/IceInst.cpp b/src/IceInst.cpp
index e4f697aa93c2822a50df186247a3c5ad15d96f7a..f336ba99b024db4c87c1c50f03d9483ec682be5a 100644
--- a/src/IceInst.cpp
+++ b/src/IceInst.cpp
@@ -493,10 +493,11 @@ InstFakeDef::InstFakeDef(Cfg *Func, Variable *Dest, Variable *Src)
     addSource(Src);
 }
 
-InstFakeUse::InstFakeUse(Cfg *Func, Variable *Src)
-    : InstHighLevel(Func, Inst::FakeUse, 1, nullptr) {
+InstFakeUse::InstFakeUse(Cfg *Func, Variable *Src, uint32_t Weight)
+    : InstHighLevel(Func, Inst::FakeUse, Weight, nullptr) {
   assert(Src);
-  addSource(Src);
+  for (uint32_t i = 0; i < Weight; ++i)

[John, 2015/12/22 15:44:38]

What's the difference between this implementation, and doing

Src->setWeight(Src->getWeight() + Weight);

?

[Jim Stichnoth, 2015/12/28 07:54:07]

There could be several instances of FakeUse, pointing to the same Src, but e.g.
at different loop nesting levels.  Setting the weight of the unique Src wouldn't
accurately capture that.

+    addSource(Src);
 }
 
 InstFakeKill::InstFakeKill(Cfg *Func, const Inst *Linked)