[PATCH] D27157: IRGen: Remove all uses of CreateDefaultAlignedLoad.

John McCall via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Mon Nov 28 14:30:34 PST 2016


rjmccall added inline comments.


================
Comment at: clang/lib/CodeGen/CGBuilder.h:126
   // FIXME: these "default-aligned" APIs should be removed,
   // but I don't feel like fixing all the builtin code right now.
   llvm::StoreInst *CreateDefaultAlignedStore(llvm::Value *Val,
----------------
pcc wrote:
> rjmccall wrote:
> > Oh, please remove this comment, too, since you've now achieved it. :)
> We still have CreateDefaultAlignedStore though.
Oh, yes, of course.  In that case, this LGTM.


https://reviews.llvm.org/D27157





More information about the cfe-commits mailing list