[llvm-commits] [llvm] r50869 - /llvm/trunk/lib/VMCore/Instruction.cpp

[Dan Gohman]

Hi Chris,

Should mayReadFromMemory consider that a volatile store
reads memory, similar to how mayWriteToMemory considers
that a volatile load may write to memory?

Dan

On May 8, 2008, at 2:47 PM, Chris Lattner wrote:

> Author: lattner
> Date: Thu May  8 16:47:43 2008
> New Revision: 50869
>
> URL: http://llvm.org/viewvc/llvm-project?rev=50869&view=rev
> Log:
> store can't read from memory.
>
> Modified:
>    llvm/trunk/lib/VMCore/Instruction.cpp
>
> Modified: llvm/trunk/lib/VMCore/Instruction.cpp
> URL: http://llvm.org/viewvc/llvm-project/llvm/trunk/lib/VMCore/Instruction.cpp?rev=50869&r1=50868&r2=50869&view=diff
>
> =
> =
> =
> =
> =
> =
> =
> =
> ======================================================================
> --- llvm/trunk/lib/VMCore/Instruction.cpp (original)
> +++ llvm/trunk/lib/VMCore/Instruction.cpp Thu May  8 16:47:43 2008
> @@ -225,7 +225,6 @@
>   switch (getOpcode()) {
>   default: return false;
>   case Instruction::Free:
> -  case Instruction::Store:
>   case Instruction::VAArg:
>     return true;
>   case Instruction::Call:
>
>
> _______________________________________________
> llvm-commits mailing list
> llvm-commits at cs.uiuc.edu
> http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits