[PATCH] D146872: [1/N][IR] Permit load/store/alloca for struct of the same scalable vector type

Yueh-Ting (eop) Chen via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Apr 5 05:39:36 PDT 2023


eopXD added a comment.

Also addressed comments from Kito:

- Added description in Language Reference Manual for the relaxation added in this patch.



================
Comment at: llvm/lib/CodeGen/SelectionDAG/SelectionDAG.cpp:6966
                                            const SDNodeFlags Flags) {
+  if (Offset.getKnownMinValue() == 0)
+    return Base;
----------------
craig.topper wrote:
> Why is this needed?
No this is not needed here. The original intent was to return base when a scalable structure was queried here.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D146872



More information about the llvm-commits mailing list