[PATCH] D14699: [GlobalOpt] Allow constant globals to be SRA'd

James Molloy via llvm-commits llvm-commits at lists.llvm.org
Mon Nov 16 08:18:53 PST 2015


jmolloy added inline comments.

================
Comment at: test/Transforms/GlobalOpt/globalsra.ll:31
@@ +30,3 @@
+define void @onlystore2() {
+        store i32 123, i32* getelementptr ({ i32, float, { double } }, { i32, float, { double } }* @G2, i32 0, i32 0)
+        ret void
----------------
joker.eph wrote:
> It is strange that we can store to a constant?
What Danny said - also, these tests are carbon copies of the existing ones above, they just  operate on a constant instead of a global.


Repository:
  rL LLVM

http://reviews.llvm.org/D14699





More information about the llvm-commits mailing list