[PATCH] D43323: [NFC] Consolidate six getPointerOperand() utility functions into one place

Hideki Saito via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Feb 16 11:04:26 PST 2018


hsaito added a comment.

In https://reviews.llvm.org/D43323#1009734, @ashutosh.nema wrote:

> Mostly looks OK, just few minor comment.


Will do.

Indentation fix is for line 5570, right? Just making sure.

Whose LGTM should I get to change Instructions.h or to create InstructionUtils.h? I'll invite Chris/Chandler for review, but I understand they are super busy with so many things.....


Repository:
  rL LLVM

https://reviews.llvm.org/D43323





More information about the llvm-commits mailing list