[PATCH] D150373: [WIP][RISCV] Avoid early resolution of symbol differences

[Philip Reames via Phabricator via llvm-commits]

reames created this revision.
Herald added subscribers: jobnoorman, luke, VincentWu, vkmr, frasercrmck, evandro, luismarques, apazos, sameer.abuasal, s.egerton, Jim, benna, psnobl, jocewei, PkmX, the_o, brucehoult, MartinMosbeck, rogfer01, edward-jones, zzheng, jrtc27, shiva0217, kito-cheng, niosHD, sabuasal, bollu, simoncook, johnrusso, rbar, asb, hiraditya, arichardson, mcrosier, emaste.
Herald added a project: All.
reames requested review of this revision.
Herald added subscribers: llvm-commits, pcwang-thead, eopXD.
Herald added a project: LLVM.

This is an alternative to D150004 <https://reviews.llvm.org/D150004>, mostly to prompt discussion.  @MaskRay pointed out @compnerd's previous change here, and I noticed a bit of code being removed which seemed to be relevant to the evaluateAsAbsolute discussion.  As I'd suspected, adding this back in does seem to fix the specific bugs we have test coverage for.

I want to be careful here and say this is not a real patch.  I'm solidly out of my area, and wouldn't feel comfortable landing this.  This is purely to see if it helps inform the discussion on the other patch.


https://reviews.llvm.org/D150373

Files:
  llvm/lib/MC/MCExpr.cpp
  llvm/test/MC/ELF/RISCV/gen-dwarf.s

-------------- next part --------------
A non-text attachment was scrubbed...
Name: D150373.521316.patch
Type: text/x-patch
Size: 4525 bytes
Desc: not available
URL: <http://lists.llvm.org/pipermail/llvm-commits/attachments/20230511/57e3253f/attachment.bin>