[PATCH] [ELF] Initial support for symbols visibility

[Shankar Kalpathi Easwaran]

I dont think we need a separate visibility field in DefinedAtoms. The visibility is already modeled as part of Scope.

<snip>

  /// The scope in which this atom is acessible to other atoms.
  enum Scope {
  ¦ scopeTranslationUnit,  ///< Accessible only to atoms in the same translation
  ¦ ¦ ¦ ¦ ¦ ¦ ¦ ¦ ¦ ¦      ///  unit (e.g. a C static).
  ¦ scopeLinkageUnit,      ///< Accessible to atoms being linked but not visible
  ¦ ¦ ¦ ¦ ¦ ¦ ¦ ¦ ¦ ¦      ///  to runtime loader (e.g. visibility=hidden).
  ¦ scopeGlobal            ///< Accessible to all atoms and visible to runtime
  ¦ ¦ ¦ ¦ ¦ ¦ ¦ ¦ ¦ ¦      ///  loader (e.g. visibility=default).
  };

</end snip>


REPOSITORY
  rL LLVM

http://reviews.llvm.org/D8325

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/