[PATCH] D51327: [MemorySSA] Add expesive check for validating clobber accesses.

Alina Sbirlea via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Tue Aug 28 14:51:20 PDT 2018


asbirlea marked 4 inline comments as done.
asbirlea added inline comments.


================
Comment at: lib/Analysis/MemorySSA.cpp:1710
+/// Check sanity of the clobbering instruction for I.
+void MemorySSA::checkClobberSanityInstruction(Instruction *I) {
+  ::checkClobberSanityInstruction(I, *this);
----------------
george.burgess.iv wrote:
> This appears unused?
I was using this locally in a unittest, but it doesn't make sense to have unittests assert, so removing it.


Repository:
  rL LLVM

https://reviews.llvm.org/D51327





More information about the llvm-commits mailing list