[PATCH] D61911: [GlobalOpt] Allow dead struct fields in SRA with non constant offset.

Christian Bruel via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Jul 1 02:19:32 PDT 2019


chrib abandoned this revision.
chrib added a comment.

the declaration from Bug #38309 currently generates

@g_data = dso_local local_unnamed_addr global [8 x i16] [i16 1, i16 1, i16 1, i16 1, i16 0, i16 0, i16 0, i16 0], align 2

which seems better than a Struct LLVM IR construct as in :
.g_data = internal global <{ [8 x i16], [8 x i16] }> ...

So if my assumption that the test globalsra-multigep.ll was badly constructed is wrong, there don't seem I can fix the semantic inconsistency with the C struct UB without inconsistencies with this regression test.

sorry for the noise, abandoning this proposal


Repository:
  rL LLVM

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D61911/new/

https://reviews.llvm.org/D61911





More information about the llvm-commits mailing list