[PATCH] D28898: [ELF] - Allow emulation to be different from input objects target.

George Rimar via llvm-commits llvm-commits at lists.llvm.org
Mon Jan 23 07:48:35 PST 2017


>> +  // Take machine type from emulation first. That is done early so
>> +  // we are able to report wrong emulation values if any.
>> +  if (!Config->Emulation.empty()) {
>> +    std::tie(Config->EKind, Config->EMachine, Config->OSABI) =
>> +        parseEmulation(Config->Emulation);
>> +    Config->MipsN32Abi = Config->Emulation == "elf32btsmipn32" ||
>> +                         Config->Emulation == "elf32ltsmipn32";
>> +  }
>
>What breaks without this? I am probably OK with completely ignoring -m
>if a .o is present (which I assume is what gold does), but it feels odd
>to use only some bits of it.
>
>Cheers,
>Rafael

Sorry, I read question wrong at first.
If we completely ignore -m that should work fine I think.
But we will be unable to catch wrong values for -m if any.
like -m blah.

If it is OK, we can do that.

George.


More information about the llvm-commits mailing list