[PATCH] D40447: [CGP] Fix handling of null pointer values in optimizeMemoryInst

[Serguei Katkov via Phabricator via llvm-commits]

skatkov accepted this revision.
skatkov added a comment.
This revision is now accepted and ready to land.

Agreed. I also came up to the same case (null is not considered as non-trivial) however not sure it is a cause of hangs-up due to select is off by default in LLVM trunk at this moment. But probably we have the same issue with Phi.

There is also one issue related to initialMap due to it seems it is possible that two bases can have different types but we crash with assert in this case... Will try to come up with a patch today.


Repository:
  rL LLVM

https://reviews.llvm.org/D40447