Re: [PATCH v2] gas: fix MAX_ISTACK for loopgtz L32R widening
Project / Subsystem
binutils / gas
Date
2026-08-31
Proposer
Max Filippov <jcmvbkbc@gmail.com>
Source type
public_inbox
Consensus
Committed
Sentiment
—/10
Technical tradeoffs
- • Increases memory usage for the instruction stack.
- • Fixes a crash/overflow issue, improving robustness.
All attributes
- project
- binutils
- subsystem
- gas
- patch_id
- —
- discussion_id
- CAMo8BfJkN2jh5VLuEr0US2Ed+Hsv0PcD_nd_G7u6Ti_SfEL7+g@mail.gmail.com
- source_type
- public_inbox
- title
- Re: [PATCH v2] gas: fix MAX_ISTACK for loopgtz L32R widening
- headline
- gas: fix MAX_ISTACK for loopgtz L32R widening
- tldr
- Increase MAX_ISTACK in gas for Xtensa to accommodate loopgtz relaxation.
- proposer
- Max Filippov <jcmvbkbc@gmail.com>
- consensus
- Committed
- outcome
- committed
- sentiment_score
- —
- technical_tradeoffs
-
- • Increases memory usage for the instruction stack.
- • Fixes a crash/overflow issue, improving robustness.
- series_id
- —
- series_role
- cover
- series_parts
- []
- tags
-
- • binutils
- • gas
- • xtensa
- • assembler
- • optimization
- bugzilla_url
- —
- date
- 2026-08-31T00:00:00.000Z
Re: [PATCH v2] gas: fix MAX_ISTACK for loopgtz L32R widening
A recent change in gas for Xtensa replaced loop widening with an L32R-based sequence. The loopgtz variant of this sequence requires 13 istack entries, exceeding the previous MAX_ISTACK limit of 12. This patch increases MAX_ISTACK to 16, documents the requirement, and improves overflow reporting. A new test case is included.