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?