[PATCH] D54661: [WebAssembly] Don't override default implementation of isOffsetFoldingLegal. NFC.

Sam Clegg via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Mar 15 18:56:11 PDT 2019


sbc100 added a comment.

This is needed for my PIC change: https://reviews.llvm.org/D54647

It seems to do exactly what we want.  It avoids folding the constant into access of the global address itself, but it still can fold the constant into the follow load.  e.g.: from that CL:

  global.get $push0=, g at GOT
  i32.load  $push1=, 40($pop0)




Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D54661





More information about the llvm-commits mailing list