[PATCH] D146987: [Assignment Tracking] Enable by default

Tue Apr 25 02:27:46 PDT 2023

Orlando added a comment.

Thanks everyone for your reports and help, it's really appreciated.

In D146987#4286318 <https://reviews.llvm.org/D146987#4286318>, @dmgreen wrote:

> Hello, I also noticed this potentially causing problems for scalable vectors:
> https://godbolt.org/z/qdr8P86aW
> That probably counts as one of the "edge cases for things we hadn't accounted for".
> Thanks

You're right there. Fixed the assertion firing in D149137 <https://reviews.llvm.org/D149137>, but the behaviour isn't quite right yet so I've opened a bug for it here llvm.org/PR62346 <https://llvm.org/PR62346>.

---

@mstorsjo and @srj, the issue with the faulty assertion in a std::sort predicate is fixed with D149045 <https://reviews.llvm.org/D149045>.

---

I found an additional assertion failure on the chromium build page (for pigweed) which I've fixed in D149135 <https://reviews.llvm.org/D149135>.

Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D146987