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

src/IceELFSection.h

Index: src/IceELFSection.h
diff --git a/src/IceELFSection.h b/src/IceELFSection.h
index 6d24e4af5e13ba3d060ecdb7186c2f8c3fed2499..369c14d07d9aae9a0a430dfb94b41e7f988046f5 100644
--- a/src/IceELFSection.h
+++ b/src/IceELFSection.h
@@ -352,10 +352,11 @@ void ELFRelocationSection::writeData(const GlobalContext &Ctx, ELFStreamer &Str,
                                      const ELFSymbolTableSection *SymTab) {
   for (const AssemblerFixup &Fixup : Fixups) {
     const ELFSym *Symbol;
+    constexpr Assembler *Asm = nullptr;

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

Why not declaring this inside the scope where it is used? I find useful to have
declarations only within the scope where they are needed.

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

Done.

     if (Fixup.isNullSymbol())
       Symbol = SymTab->getNullSymbol();
     else
-      Symbol = SymTab->findSymbol(Fixup.symbol(&Ctx));
+      Symbol = SymTab->findSymbol(Fixup.symbol(&Ctx, Asm));
     if (!Symbol)
       llvm::report_fatal_error("Missing symbol mentioned in reloc");