Revert "[GPU] Keep RMS normalization nodes in FP32 to prevent NaN from FP16 overflow (#36504)"#36892
Conversation
…m FP16 overflow (openvinotoolkit#36504)" This reverts commit 5297758.
geunhwan
left a comment
There was a problem hiding this comment.
For resolving performance regression in many models.
CI Doctor — Merge Queue failure on this PRPipeline: Linux (Ubuntu 22.04, Python 3.11) Possible remedy
What happenedThe Docker job failed because the runner could not fetch an OAuth token from the internal Azure Container Registry mirror (
|
CI Doctor — Merge Queue failure on this PRPipeline: Linux (Ubuntu 22.04, ARM64 cross-compilation, Python 3.11) Possible remedy
What happenedThe
|
CI Doctor — Merge Queue failure on this PRPipeline: Linux (Ubuntu 22.04, ARM64 cross-compilation, Python 3.11) Possible remedy
What happenedThe CPU functional tests job failed on a single test:
|
Details:
Root Cause
PR #36504 was introduced to fix NaN in ParlerTTS Mini (CVS-187992) by keeping RMS nodes in FP32. However, it unconditionally marks ALL RMS nodes across all models, adding FP16↔FP32 conversion overhead on every transformer layer.
Tickets: