[PATCH] D24037: [LTO] Fix the logic for dropping unnamed_addr

Davide Italiano via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 30 11:34:53 PDT 2016


davide added a comment.

Found the thinko, it should be

  // Merge in the new unnamed_addr attribute.
  if (WasInserted)
    S->HasUnnamedAddr = HasUnnamedAddr;
  else
    S->HasUnnamedAddr &= HasUnnamedAddr;

Otherwise HasUnnamedAddr is not set correctly in the case you pointed out. Updating


https://reviews.llvm.org/D24037





More information about the llvm-commits mailing list