This is an archive of the discontinued LLVM Phabricator instance.

Cache LSDA label for future use (NFC)
AbandonedPublic

Authored by aheejin on Sep 28 2018, 6:02 PM.

Details

Reviewers
rnk
majnemer
Summary

This adds setLSDALabel and getLSDALabel to EHStreamer, to cache
per-function LSDA starting label for future use. Currently no EH schemes
need this, but wasm EH scheme needs to this in a follow-up patch, and
because this LSDA label is emitted in a target-independent part of code,
it is not easy to make it as a wasm-only function.

Diff Detail

Event Timeline

aheejin created this revision.Sep 28 2018, 6:02 PM
aheejin added a comment.EditedSep 28 2018, 6:04 PM

I couldn't add setLSDALabel to [[ https://github.com/llvm-mirror/llvm/blob/993ef0ca960f8ffd107c33bfbf1fd603bcf5c66c/lib/CodeGen/AsmPrinter/WinException.cpp#L1063 | emitCLRExceptionTable ]], because to me it was unclear which symbol is the per-function LSDA symbol. Please let me know if there is one.

aheejin abandoned this revision.Sep 29 2018, 12:59 AM

Hmm, this does not look like a good approach anyway. Sorry!