[PATCH] D33284: [CodeGen] Propagate LValueBaseInfo instead of AlignmentSource

[Krzysztof Parzyszek via Phabricator via cfe-commits]

kparzysz updated this revision to Diff 99353.
kparzysz marked 2 inline comments as done.
kparzysz added a comment.

Changed the getNatural.*TypeAlignment to produce the entire LValueBaseInfo, assuming MayAlias to be false.

Added merging of LValueBaseInfos.  The merging assumes that the alignment source in the parameter overrides the existing source.  That works for the existing cases (cast), but I'm not sure if that's correct in general.  Is there is an implicit ordering of alignment sources?


Repository:
  rL LLVM

https://reviews.llvm.org/D33284

Files:
  lib/CodeGen/CGAtomic.cpp
  lib/CodeGen/CGBlocks.cpp
  lib/CodeGen/CGBuiltin.cpp
  lib/CodeGen/CGClass.cpp
  lib/CodeGen/CGExpr.cpp
  lib/CodeGen/CGExprCXX.cpp
  lib/CodeGen/CGObjC.cpp
  lib/CodeGen/CGObjCRuntime.cpp
  lib/CodeGen/CGOpenMPRuntime.cpp
  lib/CodeGen/CGStmtOpenMP.cpp
  lib/CodeGen/CGValue.h
  lib/CodeGen/CodeGenFunction.cpp
  lib/CodeGen/CodeGenFunction.h

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D33284.99353.patch
Type: text/x-patch
Size: 61313 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/cfe-commits/attachments/20170517/fcff0c86/attachment-0001.bin>