Refactor X86 symbol access classification

Reid Kleckner via llvm-commits llvm-commits at lists.llvm.org
Thu May 19 15:40:44 PDT 2016


Looks good, looking forward to pulling duplicated code that
effectively computes shouldAssumeDSOLocal out of the other targets.

On Thu, May 19, 2016 at 3:26 PM, Rafael EspĂ­ndola
<llvm-commits at lists.llvm.org> wrote:
> Rebased patch attached.
>
> On 19 May 2016 at 15:50, Rafael EspĂ­ndola <rafael.espindola at gmail.com> wrote:
>> Refactor X86 symbol access classification.
>>
>> This refactors the logic in X86 to avoid code duplication. It also
>> splits it in two steps: it first decides if a symbol is local to the
>> DSO and then uses that information to decide how to access it.
>>
>> The first part is implemented by shouldAssumeDSOLocal. It is not in
>> any way specific to X86. In a followup patch I intend to move it to
>> somewhere common and reused it in other backends.
>>
>> Cheers,
>> Rafael
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at lists.llvm.org
> http://lists.llvm.org/cgi-bin/mailman/listinfo/llvm-commits
>


More information about the llvm-commits mailing list