[llvm] [AssignemntTracking] Skip large types in redundant debug info pruning (PR #74329)

J. Ryan Stinnett via llvm-commits llvm-commits at lists.llvm.org
Mon Dec 4 09:29:00 PST 2023


================
@@ -2296,8 +2296,11 @@ removeRedundantDbgLocsUsingBackwardScan(const BasicBlock *BB,
           getAggregate(FnVarLocs.getVariable(RIt->VariableID));
       uint64_t SizeInBits = Aggr.first->getSizeInBits().value_or(0);
 
-      if (SizeInBits == 0) {
+      const uint64_t MaxSize = std::numeric_limits<unsigned>::max() - 63;
----------------
jryans wrote:

Is there some constant etc. from the `BitVector` code that could be reused here to compute this `63` value? It feels like a random magic number at the moment.

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


More information about the llvm-commits mailing list