[PATCH] D89341: [X86] Encode global symbol address in sub if possible

Wei Wang via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Oct 13 17:31:38 PDT 2020


weiwang added a comment.

Thanks for the comment, Craig.

`isSExtAbsoluteSymbolRef ` does seem to check for the width of immediate. If the immediate can be encoded directly, the node should be replaced with one of the `SUB64ri*` nodes. I think the `X86Wrapper` node can be replaced with a corresponding `imm` node if conditions are met, then the matching can proceed.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D89341



More information about the llvm-commits mailing list