[PATCH] D26581: Make WholeProgramDevirt understand ConstStruct vtables.

[l via Phabricator via llvm-commits]

LemonBoy marked an inline comment as done.
LemonBoy added inline comments.


================
Comment at: lib/Transforms/IPO/WholeProgramDevirt.cpp:386
 
+Optional<unsigned>
+DevirtModule::getOpAtOffset(const Constant *I, const uint64_t Offset) {
----------------
pcc wrote:
> Could be simpler to return a `const Constant *` from this function.
I suppose you meant `const Value *` since that's what `getOperand` returns.


https://reviews.llvm.org/D26581