[PATCH] D58982: DAG: allow DAG pointer size different from memory representation.

Sun Jan 23 05:00:29 PST 2022

cad.soledad added inline comments.

================
Comment at: llvm/lib/CodeGen/SelectionDAG/SelectionDAGBuilder.cpp:3922
       // it.
       IdxN = DAG.getSExtOrTrunc(IdxN, dl, N.getValueType());

----------------
Hi, I have a little question here.
The pointer is 32bits in memory and 64bits in SelectionDag.If the offset’s sign bit is 1, for example:0xfb567000,it will extend to 0xfffffffffb567000, so it will lead crash.

Repository:
  rL LLVM

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

https://reviews.llvm.org/D58982