[PATCH] ELF: Fix defintion of STT_LOOS and STT_HIOS.

Rafael EspĂ­ndola rafael.espindola at gmail.com
Tue Jun 23 04:27:07 PDT 2015


LGTM

On 23 June 2015 at 07:22, Tom Stellard <thomas.stellard at amd.com> wrote:
> Hi chandlerc, Bigcheese, rafael,
>
> http://reviews.llvm.org/D10647
>
> Files:
>   include/llvm/Support/ELF.h
>
> Index: include/llvm/Support/ELF.h
> ===================================================================
> --- include/llvm/Support/ELF.h
> +++ include/llvm/Support/ELF.h
> @@ -823,9 +823,9 @@
>    STT_FILE    = 4,   // Local, absolute symbol that refers to a file
>    STT_COMMON  = 5,   // An uninitialized common block
>    STT_TLS     = 6,   // Thread local data object
> -  STT_LOOS    = 7,   // Lowest operating system-specific symbol type
> -  STT_HIOS    = 8,   // Highest operating system-specific symbol type
>    STT_GNU_IFUNC = 10, // GNU indirect function
> +  STT_LOOS    = 10,  // Lowest operating system-specific symbol type
> +  STT_HIOS    = 12,  // Highest operating system-specific symbol type
>    STT_LOPROC  = 13,  // Lowest processor-specific symbol type
>    STT_HIPROC  = 15   // Highest processor-specific symbol type
>  };
>
> EMAIL PREFERENCES
>   http://reviews.llvm.org/settings/panel/emailpreferences/



More information about the llvm-commits mailing list