[PATCH] D149006: [llvm][Support] Replace `%` operator with `&` in `Align::isAligned(...)`

Florian Hahn via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Apr 28 02:32:11 PDT 2023


fhahn added a comment.

> it is easier to check reminder by conjunction with mask, which is (Pow2Value - 1).

Hmm, easier in what respect? Isn't the original code more straight-forward?


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D149006



More information about the cfe-commits mailing list