[PATCH] D109714: Reduce maximum width of _ExtInt from 2^24-1 to 2^23 to workaround SelectionDAG issue.

Erich Keane via Phabricator via llvm-commits llvm-commits at lists.llvm.org
Mon Sep 13 12:31:55 PDT 2021


erichkeane accepted this revision.
erichkeane added a comment.
This revision is now accepted and ready to land.

Changes to the tests are less intense than I would have thought!  In general I'm Ok with this, I don't think anyone would really notice/mind this all that much, and since we can't actually implement this in the backends, it doesn't really make sense to have some sort of flag for this.  I have a slight bit of sympathy for the backends that could consume this, but I'd think we need to have our implementation-limit be the least-common-denominator of our backends anyway.


Repository:
  rG LLVM Github Monorepo

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

https://reviews.llvm.org/D109714



More information about the llvm-commits mailing list