[PATCH] D68488: [PATCH 03/27] [noalias] [IR] Introduce ptr_provenance for LoadInst/StoreInst

[Roman Lebedev via Phabricator via llvm-commits]

lebedev.ri added inline comments.


================
Comment at: llvm/include/llvm/IR/Instructions.h:210
 
+  ~LoadInst() {
+    setLoadInstNumOperands(2); // needed by operator delete
----------------
Are there `Instruction`/`Value` subclasses already, that define non-trivial destructor?
I think this may be a problem for bumpptr allocators.


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

https://reviews.llvm.org/D68488