[PATCH] D53556: [ExecutionEngine] Fix out-of-bounds access in the interpreter

[Eugene Sharygin via Phabricator via llvm-commits]

eush added inline comments.


================
Comment at: test/ExecutionEngine/Interpreter/rand.ll:1
+; RUN: lli -O0 -force-interpreter < %s
+
----------------
nicholas wrote:
> This matches the style in intrinsics.ll, but alias.ll uses "%lli -force-interpreter %s", the differences being that it uses %lli instead of lli and does not use < redirection. I do not know which style is correct or preferred.
I created https://llvm.org/PR39524 about using `lli` vs `%lli` in tests. There is no resolution on that yet.


Repository:
  rL LLVM

https://reviews.llvm.org/D53556