[PATCH] D48131: [RISCV] Implement codegen for cmpxchg on RV32I

[Alex Bradbury via Phabricator via llvm-commits]

asb added inline comments.


================
Comment at: lib/Target/RISCV/RISCVISelLowering.cpp:1744
+  Value *Addr = CI->getPointerOperand();
+  Value *AddrInt = Builder.CreatePtrToInt(Addr, Builder.getInt32Ty());
+  Value *AlignedAddr = Builder.CreateIntToPtr(
----------------
t.p.northover wrote:
> Is this an intentional choice not to support 64-bit pointers? Even if so, an assert might be good.
RV64I isn't upstreamed yet. Most other parts of the atomics handling have asserts, and I should definitely add one here. I could try and have it do the right thing, but I think asserting is better than committing code that hasn't been tested. Good catch, thanks.


https://reviews.llvm.org/D48131