[PATCH] D11043: Const fold vtable load from global variable

Philip Reames listmail at philipreames.com
Wed Jul 8 22:53:58 PDT 2015


reames added a subscriber: reames.
reames added a comment.

General comment: Having a special case in TBAA for a constant after initialization value seems like the wrong factoring here.  Supporting a more general extensions of !invariant.load would seem to be more generally useful.

Having said that, it doesn't look like you're adding the mechanism, just extending it.  As such, my comment isn't directly relevant to the patch at hand.


Repository:
  rL LLVM

http://reviews.llvm.org/D11043







More information about the llvm-commits mailing list