[PATCH] D69639: Add support for lowering 32-bit/64-bit pointers

Matt Arsenault via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Wed Oct 30 14:21:28 PDT 2019


arsenm added inline comments.


================
Comment at: llvm/lib/Target/X86/X86ISelLowering.cpp:2396-2397
+  const TargetMachine &TM = getTargetMachine();
+  if ((isPtrSizeOrDefaultAddrSpace(SrcAS) &&
+       isPtrSizeOrDefaultAddrSpace(DestAS)) &&
+      TM.getPointerSize(SrcAS) != TM.getPointerSize(DestAS))
----------------
These look redundant with the size check to me


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D69639





More information about the llvm-commits mailing list