[PATCH] D108303: [clang][openmp] Disable embedded elf notes

2021-08-18 Thread Vyacheslav Zakharin via Phabricator via cfe-commits
vzakhari added a comment. In D108303#2952393 , @JonChesterfield wrote: > I like D108246 more. None of the > offloading tests updated in D108246 failed > with the above patch, perhaps

[PATCH] D108303: [clang][openmp] Disable embedded elf notes

2021-08-18 Thread Jon Chesterfield via Phabricator via cfe-commits
JonChesterfield abandoned this revision. JonChesterfield added a comment. I like D108246 more. None of the offloading tests updated in D108246 failed with the above patch, perhaps they're not run by `make check-openmp` Repos

[PATCH] D108303: [clang][openmp] Disable embedded elf notes

2021-08-18 Thread Vyacheslav Zakharin via Phabricator via cfe-commits
vzakhari added a comment. Hi Jon, I am about to merge D108246 that disables the notes embedding. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D108303/new/ https://reviews.llvm.org/D108303 ___

[PATCH] D108303: [clang][openmp] Disable embedded elf notes

2021-08-18 Thread Jon Chesterfield via Phabricator via cfe-commits
JonChesterfield created this revision. JonChesterfield added reviewers: vzakhari, ABataev, grokos, sdmitriev, jdoerfert, ronlieb. Herald added subscribers: guansong, yaxunl. JonChesterfield requested review of this revision. Herald added subscribers: cfe-commits, sstefan1. Herald added a project: