[clang] [LifetimeSafety] Revisit tracking moved objects (PR #178670)

[via cfe-commits]

================
@@ -63,6 +63,7 @@ static llvm::BitVector computePersistentOrigins(const 
FactManager &FactMgr,
              Cur = Cur->peelOuterOrigin())
           CheckOrigin(Cur->getOuterOriginID());
         break;
+      case Fact::Kind::MovedOrigins:
----------------
graphite-app[bot] wrote:

Enum value mismatch: Using `Fact::Kind::MovedOrigins` (plural) but the enum is 
defined as `Fact::Kind::MovedOrigin` (singular) in Facts.h line 48. This case 
will never match, causing `MovedOriginFact` instances to not be properly 
handled in loan propagation.

```cpp
// Should be:
case Fact::Kind::MovedOrigin:
```
```suggestion
      case Fact::Kind::MovedOrigin:
```
  

*Spotted by [Graphite 
Agent](https://app.graphite.com/diamond/?org=llvm&ref=ai-review-comment)*<i 
class='graphite__hidden'><br /><br /><a 
href="https://app.graphite.com/github/pr/llvm/llvm-project/178670?chatWithGeneratedComment=9dd8e20c-5352-4d3e-92a8-e86afee05734";><picture><source
 media="(prefers-color-scheme: dark)" 
srcset="https://static.graphite.dev/github-diamond-fix-in-graphite-dark.svg";><source
 media="(prefers-color-scheme: light)" 
srcset="https://static.graphite.dev/github-diamond-fix-in-graphite-light.svg";><img
 alt="Fix in Graphite" 
src="https://static.graphite.dev/github-diamond-fix-in-graphite-dark.svg";></picture></a></i><i
 class='graphite__hidden'><br /><br />Is this helpful? React 👍 or 👎 to let us 
know.</i>

https://github.com/llvm/llvm-project/pull/178670
_______________________________________________
cfe-commits mailing list
cfe-commits@lists.llvm.org
https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits