[llvm] r222760 - Reapply 222538 and update tests to explicitly request small code model

Joerg Sonnenberger joerg at britannica.bec.de
Thu Nov 27 08:16:38 PST 2014


On Thu, Nov 27, 2014 at 11:01:54AM -0500, Rafael EspĂ­ndola wrote:
> This is odd at least.
> 
> First,  -code-model=smal is the default, please don't specify it everywhere.
> Second, why the PIC change? The gnu assembler always uses a
> DW_EH_PE_pcrel encoding. What is the advantage of not doing that?

I specified it explicitly to get a consistent value. I'm not sure pcrel
buys anything for non-PIC, but using absolute locations is certainly
more consistent with what we do for CFI.

Joerg



More information about the llvm-commits mailing list