Details
Details
- Reviewers
aaron.ballman erichkeane shafik cor3ntin
Diff Detail
Diff Detail
Event Timeline
| clang/lib/AST/Interp/InterpBuiltin.cpp | ||
|---|---|---|
| 232 | Yes, this is also covered by https://reviews.llvm.org/D155568 | |
Comment Actions
LGTM
| clang/lib/AST/Interp/InterpBuiltin.cpp | ||
|---|---|---|
| 232 | Ah, bitten by the massive patch stack again. :-) | |
Comment Actions
Also forgot the link in the commit message here. This is closed by https://github.com/llvm/llvm-project/commit/6ba4b213346fad36634a5571feda4f4481097c3a
Should we add an assert that [1] is a valid index to read from?