diff options
| author | Tomas Matheson <tomas.matheson@arm.com> | 2023-07-08 23:51:32 +0100 |
|---|---|---|
| committer | Tomas Matheson <tomas.matheson@arm.com> | 2023-07-09 10:14:48 +0100 |
| commit | 6d4a3e917c1e32b665e1177c92f6b90032817feb (patch) | |
| tree | 5621866f85406980f2027481e879b893f7a9b43c /llvm/lib/CodeGen/CodeGenPrepare.cpp | |
| parent | 4c66b0325905bdb367a5453574c10e535b900b5e (diff) | |
| download | llvm-6d4a3e917c1e32b665e1177c92f6b90032817feb.zip llvm-6d4a3e917c1e32b665e1177c92f6b90032817feb.tar.gz llvm-6d4a3e917c1e32b665e1177c92f6b90032817feb.tar.bz2 | |
Revert "[TableGen] Use heap allocated arrays instead of vectors for TreePatternNode::Types and ResultPerm. NFC"
While working on DAGISelMatcherEmitter I've hit several runtime errors
caused by accessing TreePatternNode::Types out of bounds. These were
difficult to debug because the switch from std::vector to unique_ptr
removes bounds checking.
I don't think the slight reduction in class size is worth the extra
debugging and memory safety problems, so I suggest we revert this.
This reverts commit d34125a1a825208b592cfa8f5fc3566303d691a4.
Differential Revision: https://reviews.llvm.org/D154781
Diffstat (limited to 'llvm/lib/CodeGen/CodeGenPrepare.cpp')
0 files changed, 0 insertions, 0 deletions