[llvm-commits] [llvm] r89951 - in /llvm/trunk: lib/Analysis/BasicAliasAnalysis.cpp lib/Target/README.txt test/Analysis/BasicAA/gep-alias.ll

Duncan Sands baldrick at free.fr
Thu Nov 26 09:29:30 PST 2009


Hi Chris,

> +      Scale *= IndexScale.getZExtValue();
> +      BaseOffs += IndexOffset.getZExtValue()*Scale;

shouldn't these two lines be swapped?  I think in the BaseOffs line
you should be multiplying by the value of Scale before IndexScale was
factored into it, not afterwards...

Ciao,

Duncan.



More information about the llvm-commits mailing list