[PATCH] D157068: [Attributor] New attribute to identify what byte ranges are alive for an allocation

Shilei Tian via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Fri Sep 1 18:51:45 PDT 2023


tianshilei1992 added inline comments.


================
Comment at: llvm/lib/Transforms/IPO/Attributor.cpp:3588
+      {(unsigned)Instruction::Alloca}, UsedAssumedInformation);
+  (void)Success;
+  assert(Success && "Expected the check call to be successful!");
----------------
tianshilei1992 wrote:
> Better to use `[[maybe_unused]]` now.
Then there is no need `(void)Success;`.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D157068



More information about the llvm-commits mailing list