[PATCH] D143681: [llvm][test] restrict 2 GVN tests to just test GVN (NFC)

Nick Desaulniers via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Thu Feb 9 14:40:52 PST 2023


nickdesaulniers added inline comments.


================
Comment at: llvm/test/Transforms/GVN/no_speculative_loads_with_asan.ll:6
 
+; load of %i8 is redundant wrt. load of %i3.
 define i32 @TestNoAsan() {
----------------
Q: isn't `%i8` based on `gep ptr %i, 1` while `%i3` is based on `get ptr %i, 2`? Aren't those different addresses? Or is it because it's a 16b load (so the two adjacent 8bits)?


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D143681/new/

https://reviews.llvm.org/D143681



More information about the llvm-commits mailing list