[PATCH] D68141: [Alignment][NFC] Remove AllocaInst::setAlignment(unsigned)

Alexander Richardson via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 30 06:41:40 PDT 2019


arichardson added inline comments.


================
Comment at: llvm/trunk/lib/Target/AArch64/AArch64StackTagging.cpp:65
 
-static constexpr unsigned kTagGranuleSize = 16;
+static const Align kTagGranuleSize = Align(16);
 
----------------
Can't the Align ctor be constexpr? Will this result in a `Log2` call at runtime?


Repository:
  rL LLVM

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

https://reviews.llvm.org/D68141





More information about the llvm-commits mailing list