This is an archive of the discontinued LLVM Phabricator instance.

[clang][ifs] Dropping older experimental interface stub formats.
ClosedPublic

Authored by plotfi on Aug 21 2019, 7:00 PM.

Details

Summary

Working on a new tool, llvm-ifs, for merging interface stub files generated by clang and I've iterated on my derivative format of TBE to a newer format. This format is pretty straight forward and mainly has two fields which are the LLVM triple and the object format (since the triple doesn't encode that). For instance, you may want PE/COFF on Linux or something like that so a triple like x86_64-linux-gnu can't really tell you what the object is. Anyways, I think this format is more straight forward than something derived from objyaml and more intended for merging than TBE (also more platform agnostic).

Diff Detail

Event Timeline

plotfi created this revision.Aug 21 2019, 7:00 PM
Herald added a project: Restricted Project. · View Herald TranscriptAug 21 2019, 7:00 PM
Herald added a subscriber: cfe-commits. · View Herald Transcript
compnerd accepted this revision.Aug 22 2019, 10:17 AM

Please add a diagnostic to indicate that the provided value is invalid for the argument (or at at least extend the existing the tests to ensure that the old values are diagnosed in the driver rather hitting the assert in the frontend.

This revision is now accepted and ready to land.Aug 22 2019, 10:17 AM
plotfi updated this revision to Diff 216722.Aug 22 2019, 2:43 PM

Better diag

This revision was automatically updated to reflect the committed changes.
Herald added a project: Restricted Project. · View Herald TranscriptAug 22 2019, 4:47 PM