https://gcc.gnu.org/bugzilla/show_bug.cgi?id=69237
David Malcolm <dmalcolm at gcc dot gnu.org> changed: What |Removed |Added ---------------------------------------------------------------------------- Status|RESOLVED |UNCONFIRMED Resolution|WONTFIX |--- --- Comment #6 from David Malcolm <dmalcolm at gcc dot gnu.org> --- (In reply to Markus Trippelsdorf from comment #0) [...snip...] > In file included from ../../third_party/skia/include/core/SkPathRef.h:17:0, > from ../../third_party/skia/include/core/SkPath.h:12, > from > ../../third_party/WebKit/Source/platform/graphics/GraphicsTypes.h:32, > from > ../../third_party/WebKit/Source/platform/graphics/Image.h:33, > from > ../../third_party/WebKit/Source/core/style/StyleImage.h:31, > from > ../../third_party/WebKit/Source/core/style/NinePieceImage.h:30, > from > ../../third_party/WebKit/Source/core/style/ComputedStyle.h:38, > from > ../../third_party/WebKit/Source/core/layout/LayoutObject.h:46, > from > ../../third_party/WebKit/Source/core/animation/css/CSSAnimationUpdate.h:15, > from > ../../third_party/WebKit/Source/core/animation/css/CSSAnimations.h:37, > from > ../../third_party/WebKit/Source/core/animation/ElementAnimations.h:35, > from > ../../third_party/WebKit/Source/core/animation/ElementAnimation.h:37, > from gen/blink/bindings/core/v8/V8Element.cpp:35, > from > gen/blink/bindings/core/v8/V8GeneratedCoreBindings03.cpp:37: > ../../third_party/skia/include/core/SkTDArray.h: In member function ‘void > SkTDArray<T>::pop(T*)’: > ../../third_party/skia/include/core/SkTDArray.h:270:0: warning: statement is > indented as if it were guarded by... [-Wmisleading-indentation] > void pop(T* elem) { SkASSERT(fCount > 0); if (elem) *elem = > (*this)[fCount - 1]; --fCount; } > > > ../../third_party/skia/include/core/SkTDArray.h:270:0: note: ...this ‘if’ > clause, but it is not > > Unfortunately when adding --save-temps, the issue vanishes. Thanks for reporting this. Note that many of the locations are reported as being in column 0: in particular, for the location of the diagnostic itself. This suggests that location_t values are >= LINE_MAP_MAX_LOCATION_WITH_COLS. Hence I believe that this is a duplicate of PR preprocessor/69177 and PR c++/68819. This would be consistent with the issue going away with --save-temps.