eugenis added a comment.

The test should be in LLVM, under  test/Instrumentation/MemorySanitizer



================
Comment at: llvm/lib/Transforms/Instrumentation/MemorySanitizer.cpp:3077
+    Value *LowShadow = IRB.CreateOr(LowA, LowB);
+    Value *Shadow = IRB.CreateInsertElement(Second, LowShadow, 
IRB.getInt32(0));
+
----------------
You probably want to insert in First, not Second.

Is the generated code any better if you OR the vectors, and then shuffle to put 
the top element of First into the top element of the output? That's what LLVM 
generates if I express this logic in C.




Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D82398



_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

Reply via email to