[PATCH] D97107: Replace func name with regex in update_cc_test_checks

2021-03-12 Thread Giorgis Georgakoudis via Phabricator via cfe-commits
ggeorgakoudis updated this revision to Diff 330399. ggeorgakoudis added a comment. NFC change in update_llc_test_checks for compatibility Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97107/new/ https://reviews.llvm.org/D97107 Files: clang/test/

[PATCH] D97107: Replace func name with regex in update_cc_test_checks

2021-03-10 Thread Giorgis Georgakoudis via Phabricator via cfe-commits
ggeorgakoudis added a comment. In D97107#2617897 , @thakis wrote: > This breaks check-llvm on linux and mac: > http://45.33.8.238/linux/41398/step_12.txt > http://45.33.8.238/macm1/5291/step_10.txt > http://lab.llvm.org:8011/#/builders/109 > (and on my loc

[PATCH] D97107: Replace func name with regex in update_cc_test_checks

2021-03-10 Thread Nico Weber via Phabricator via cfe-commits
thakis added a comment. This breaks check-llvm on linux and mac: http://45.33.8.238/linux/41398/step_12.txt http://45.33.8.238/macm1/5291/step_10.txt http://lab.llvm.org:8011/#/builders/109 (and on my local build too) Please take a look, and please revert for now if it takes a while to fix. Rep

[PATCH] D97107: Replace func name with regex in update_cc_test_checks

2021-03-10 Thread Giorgis Georgakoudis via Phabricator via cfe-commits
This revision was landed with ongoing or failed builds. This revision was automatically updated to reflect the committed changes. Closed by commit rGbf58d6a1f922: Replace func name with regex in update_cc_test_checks (authored by ggeorgakoudis). Repository: rG LLVM Github Monorepo CHANGES SINC

[PATCH] D97107: Replace func name with regex in update_cc_test_checks

2021-03-10 Thread Giorgis Georgakoudis via Phabricator via cfe-commits
ggeorgakoudis updated this revision to Diff 329747. ggeorgakoudis added a comment. Add triple in test to avoid spurious failures Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97107/new/ https://reviews.llvm.org/D97107 Files: clang/test/utils/upd

[PATCH] D97107: Replace func name with regex in update_cc_test_checks

2021-03-08 Thread Giorgis Georgakoudis via Phabricator via cfe-commits
ggeorgakoudis updated this revision to Diff 329008. ggeorgakoudis added a comment. Change regex to avoid failing windows test Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97107/new/ https://reviews.llvm.org/D97107 Files: clang/test/utils/update

[PATCH] D97107: Replace func name with regex in update_cc_test_checks

2021-03-07 Thread Giorgis Georgakoudis via Phabricator via cfe-commits
ggeorgakoudis updated this revision to Diff 328845. ggeorgakoudis added a comment. Fix (another) for multiple capture groups Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97107/new/ https://reviews.llvm.org/D97107 Files: clang/test/utils/update_

[PATCH] D97107: Replace func name with regex in update_cc_test_checks

2021-03-06 Thread Giorgis Georgakoudis via Phabricator via cfe-commits
ggeorgakoudis updated this revision to Diff 328824. ggeorgakoudis added a comment. Fix for multiple capture groups Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97107/new/ https://reviews.llvm.org/D97107 Files: clang/test/utils/update_cc_test_ch

[PATCH] D97107: Replace func name with regex in update_cc_test_checks

2021-03-06 Thread Johannes Doerfert via Phabricator via cfe-commits
jdoerfert accepted this revision. jdoerfert added a comment. This revision is now accepted and ready to land. Cool, almost able to generate offloading tests :) LG Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97107/new/ https://reviews.llvm.org/D97

[PATCH] D97107: Replace func name with regex in update_cc_test_checks

2021-03-06 Thread Giorgis Georgakoudis via Phabricator via cfe-commits
ggeorgakoudis updated this revision to Diff 328823. ggeorgakoudis added a comment. Remove testing print Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97107/new/ https://reviews.llvm.org/D97107 Files: clang/test/utils/update_cc_test_checks/Inputs

[PATCH] D97107: Replace func name with regex in update_cc_test_checks

2021-03-06 Thread Giorgis Georgakoudis via Phabricator via cfe-commits
ggeorgakoudis updated this revision to Diff 328810. ggeorgakoudis added a comment. Herald added a project: clang. Herald added a subscriber: cfe-commits. Move implementation in common, add tests Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D97107/ne