[PATCH] D74311: [CodeGen] Fix the computation of the alignment of split stores.

Tue Feb 11 00:12:25 PST 2020

lebedev.ri accepted this revision.
lebedev.ri marked an inline comment as done.
lebedev.ri added a comment.

LG for real now :)

================
Comment at: llvm/test/CodeGen/PowerPC/split-store-alignment.ll:101-102
+; PPC64-NEXT:    [[TMP2:%.*]] = bitcast i64* [[P]] to i32*
+; PPC64-NEXT:    [[TMP3:%.*]] = getelementptr i32, i32* [[TMP2]], i32 1
+; PPC64-NEXT:    store i32 0, i32* [[TMP3]], align 4
+; PPC64-NEXT:    ret void
----------------
spatel wrote:
> lebedev.ri wrote:
> > Okay, i guess there is no big-endian issue after all.
> Wait - why is there no difference between LE and BE here?
> We may need to explicitly specify the datalayout on the command-line or in the test file.
Oops, thanks for catching that.
So there was an issue after all.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D74311