[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