[PATCH] [SeparateConstOffsetFromGEP] Bail there is only one GEP offset

Jingyue Wu jingyue at google.com
Tue Apr 21 09:29:56 PDT 2015


Hi James,

I don't get why

  getelementptr i8*, i8* %base, i32 %offset

is not worth splitting if `%offset` can be reassociated to variadic+const. base+variadic may still be CSE'ed in that case.


REPOSITORY
  rL LLVM

http://reviews.llvm.org/D9149

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






More information about the llvm-commits mailing list