Skip to content

Conversation

@kbobrovs
Copy link
Contributor

Simple tool for manipulating tables on files - inserting/replacing columns, adding rows, etc. To be
used to model multiple inputs/outputs in clang driver action graph node in SYCL offload compilations.

The first commit is a prerequisite and is a separate PR #1358

part of #1356

Signed-off-by: Konstantin S Bobrovsky konstantin.s.bobrovsky@intel.com

@kbobrovs
Copy link
Contributor Author

@Fznamznon, @mdtoguchi, @sndmitriev, @AGindinson - please review/approve. All review comments have been addressed.

@kbobrovs kbobrovs force-pushed the spec-const-table-tform branch 2 times, most recently from ccff2e0 to 8d7393d Compare March 23, 2020 23:45
@kbobrovs kbobrovs force-pushed the spec-const-table-tform branch from 8d7393d to 7e07365 Compare March 25, 2020 03:52
@kbobrovs
Copy link
Contributor Author

kbobrovs force-pushed the kbobrovs:spec-const-table-tform branch from 8d7393d to 7e07365

addressed review comments from @Fznamznon

@kbobrovs kbobrovs requested a review from Fznamznon March 25, 2020 04:14
Fznamznon
Fznamznon previously approved these changes Mar 25, 2020
Fznamznon
Fznamznon previously approved these changes Mar 26, 2020
@bader
Copy link
Contributor

bader commented Mar 26, 2020

@kbobrovs, could you resolve the conflict, please?

@kbobrovs
Copy link
Contributor Author

@kbobrovs, could you resolve the conflict, please?

sure, 1 sec

Signed-off-by: Konstantin S Bobrovsky <konstantin.s.bobrovsky@intel.com>
@kbobrovs
Copy link
Contributor Author

kbobrovs force-pushed the kbobrovs:spec-const-table-tform branch from 7e0dc74 to 94d3cd2 19 seconds ago

rebased onto intel/llvm/sycl

@kbobrovs kbobrovs requested a review from bader March 26, 2020 17:58
@bader bader requested a review from Fznamznon March 27, 2020 10:49
Signed-off-by: Alexey Bader alexey.bader@intel.com
@romanovvlad romanovvlad merged commit 6b44ebb into intel:sycl Mar 27, 2020
@kbobrovs kbobrovs deleted the spec-const-table-tform branch March 27, 2020 16:56
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment

Labels

None yet

Projects

None yet

Development

Successfully merging this pull request may close these issues.

4 participants