Page MenuHomePhabricator

[OpenMP][FIX] Emit debug declares only if debug info is available
ClosedPublic

Authored by jdoerfert on Jan 8 2022, 11:56 AM.

Details

Summary

The EmitDeclareOfAutoVariable introduced in D114504 and D115510 has a
precondition that cannot be violated. It is unclear if we should call it
directly given the sparse usage in clang but for now we should at least
not crash if the debug info kind is too low.

Fixes #52938.

Diff Detail

Event Timeline

jdoerfert created this revision.Jan 8 2022, 11:56 AM
jdoerfert requested review of this revision.Jan 8 2022, 11:56 AM
Herald added a project: Restricted Project. · View Herald TranscriptJan 8 2022, 11:56 AM
jdoerfert updated this revision to Diff 398358.Jan 8 2022, 11:59 AM

Clang format

ye-luo added a comment.Jan 8 2022, 1:59 PM

Confirm that #52938 is fixed by this patch.

ye-luo accepted this revision.Jan 8 2022, 1:59 PM
This revision is now accepted and ready to land.Jan 8 2022, 1:59 PM