Page MenuHomePhabricator

[LibTooling] Update Stencil to use RangeSelector
ClosedPublic

Authored by ymandel on May 20 2019, 2:14 PM.

Details

Summary

Add support for creating a StencilPart from any RangeSelector, which broadens the scope of Stencil.

Correspondingly, deprecate Stencil's specialized combinators node and sNode in favor of using the appropriate range selector directly.

Diff Detail

Repository
rL LLVM

Event Timeline

ymandel created this revision.May 20 2019, 2:14 PM
Herald added a project: Restricted Project. · View Herald TranscriptMay 20 2019, 2:14 PM
ymandel updated this revision to Diff 200747.May 22 2019, 7:58 AM

Add back node and sNode.

ymandel edited the summary of this revision. (Show Details)May 22 2019, 7:59 AM
ymandel edited the summary of this revision. (Show Details)
sbenza accepted this revision.May 22 2019, 8:42 AM
This revision is now accepted and ready to land.May 22 2019, 8:42 AM
ymandel updated this revision to Diff 200793.May 22 2019, 10:55 AM
ymandel edited the summary of this revision. (Show Details)

Adjusted name qualification and added using decls in a handful of places.

Herald added a reviewer: jfb. · View Herald Transcript
Herald added projects: Restricted Project, Restricted Project, Restricted Project. · View Herald Transcript
Herald added subscribers: llvm-commits, openmp-commits, Restricted Project and 34 others. · View Herald Transcript
ymandel updated this revision to Diff 200794.May 22 2019, 10:58 AM

Rebase onto HEAD, fixing previous diff which accidentally included other commits.

This revision was automatically updated to reflect the committed changes.