[asan] Fixed the jump to use the 4 byte offset version.

[kstoimenov]

Summary

This should have been the 4 byte version in the first place. Unfortunatelly there is no easy way to add a test as both the 1 byte and 4 byte version are printed as 'jmp' in the assembly code.

[kda]

Is there a test which require JMP_4? (I assume the 1->4 is about length of jump.)

[kstoimenov]

In D109453#2989907, @kda wrote:

Is there a test which require JMP_4? (I assume the 1->4 is about length of jump.)

I found a failure at link time for some of the binaries in Google code base. So it will require some sort of "integration test", which will try to link the code. I am not aware that such infrastructure exists in LLVM, but I might be wrong.

[kda]

I don't know, but the .ll tests get linked, so I assume there must be some
way to trigger the problem. Maybe not worth the effort.

llvm/lib/Target/X86/X86MCInstLower.cpp

void X86AsmPrinter::emitAsanReportError(Module &M, unsigned Reg,
   std::string Name = AccessInfo.IsWrite ? "store" : "load";
   MCSymbol *ReportError = OutContext.getOrCreateSymbol(
       "__asan_report_" + Name + utostr(1ULL << AccessInfo.AccessSizeIndex));
   OutStreamer->emitInstruction(MCInstBuilder(X86::MOV64rr)
                                    .addReg(X86::RDI)
                                    .addReg(X86::NoRegister + Reg),
                                STI);
   OutStreamer->emitInstruction(
-      MCInstBuilder(X86::JMP_1)
+      MCInstBuilder(X86::JMP_4)
           .addExpr(MCSymbolRefExpr::create(ReportError, MCSymbolRefExpr::VK_PLT,
                                            OutContext)),
       STI);
 }
 
 void X86AsmPrinter::emitAsanMemaccessSymbols(Module &M) {
   if (AsanMemaccessSymbols.empty())
     return;