[PATCH] D42127: [GlobalMerge] Don't merge dllexport globals

Saleem Abdulrasool via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Jan 23 09:23:46 PST 2018


compnerd added a comment.

Wouldn't it be possible to merge multiple dllexport globals?

  @x = dllexport global i32 0, align 4
  @y = dllexport global i32 0, align 4
  @z = global i32 0, align 4
  @a = global i32 0, align 4

could still merge into:

  @x = dllexport global i32 0, align 4
  @z = global i32 0, align 4

even though technically we should be able to merge all of that into a single global and preserve the dllexport attribute.


https://reviews.llvm.org/D42127





More information about the llvm-commits mailing list