[PATCH] SeparateConstOffsetFromGEP: Pass address space to isLegalAddressingMode

Jingyue Wu jingyue at google.com
Sat Jun 6 13:30:41 PDT 2015


The change itself looks good, but is it based on some WIP? TTI::isLegalAddressingMode doesn't take an address space yet.


================
Comment at: test/Transforms/SeparateConstOffsetFromGEP/R600/split-gep-and-gvn-addrspace-addressing-modes.ll:95
@@ +94,1 @@
+}
\ No newline at end of file

----------------
Please fix.

http://reviews.llvm.org/D10292

EMAIL PREFERENCES
  http://reviews.llvm.org/settings/panel/emailpreferences/






More information about the llvm-commits mailing list