[clang] [analyzer] Support interestingness in ArrayBoundV2 (PR #78315)

Balazs Benics via cfe-commits cfe-commits at lists.llvm.org
Fri Jan 19 08:33:48 PST 2024


================
@@ -33,7 +33,66 @@ using namespace taint;
 using llvm::formatv;
 
 namespace {
-enum OOB_Kind { OOB_Precedes, OOB_Exceeds, OOB_Taint };
+class StateUpdateReporter {
+  const SubRegion *Reg;
+  NonLoc ByteOffsetVal;
+  std::optional<QualType> ElementType = std::nullopt;
+  std::optional<int64_t> ElementSize = std::nullopt;
+  bool AssumedNonNegative = false;
----------------
steakhal wrote:

Maybe it's only me, but I'd prefer to put the bool flag at the end to have better storage layout (less padding bytes).
As a rule of thumb, order descending by alignment.

https://github.com/llvm/llvm-project/pull/78315


More information about the cfe-commits mailing list