[PATCH] D158254: Fixing the memory leak using split() instead of strtok

2023-08-22 Thread Harini Chilamantula via Phabricator via cfe-commits
hchilama added a comment. Please use my Intel email Chilamantula, Harini . Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D158254/new/ https://reviews.llvm.org/D158254 ___ cfe-commits mailing list cfe-com

[PATCH] D158254: Fixing the memory leak using split() instead of strtok

2023-08-22 Thread Harini Chilamantula via Phabricator via cfe-commits
hchilama added a comment. Hi @abhina.sreeskantharajan , Thanks for letting me know the process and please help me in commiting this patch. Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D158254/new/ https://reviews.llvm.org/D158254

[PATCH] D158254: Fixing the memory leak using split() instead of strtok

2023-08-21 Thread Harini Chilamantula via Phabricator via cfe-commits
hchilama added a comment. Hi @abhina.sreeskantharajan can I please know the next process on how to land this revision to opensource? Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D158254/new/ https://reviews.llvm.org/D158254 __

[PATCH] D158254: Fixing the memory leak using split() instead of strtok

2023-08-21 Thread Harini Chilamantula via Phabricator via cfe-commits
hchilama added a comment. Hi @abhina.sreeskantharajan I have verified the fix aganist the clang-format, seems fine. (FYI: I saw clean build on Friday) I have modified this object with the edit policy "All Users" Please let me know if I need to do anything else here? Repository: rG LLVM Gith

[PATCH] D158254: Fixing the memory leak using split() instead of strtok

2023-08-18 Thread Harini Chilamantula via Phabricator via cfe-commits
hchilama accepted this revision. hchilama added a comment. This revision is now accepted and ready to land. Using the split() instead of strtok to improve the performance Repository: rG LLVM Github Monorepo CHANGES SINCE LAST ACTION https://reviews.llvm.org/D158254/new/ https://reviews.llv

[PATCH] D158254: Fixing the memory leak using split() instead of strtok

2023-08-18 Thread Harini Chilamantula via Phabricator via cfe-commits
hchilama created this revision. hchilama created this object with edit policy "Administrators". Herald added a project: All. hchilama requested review of this revision. Herald added subscribers: cfe-commits, MaskRay. Herald added a project: clang. Repository: rG LLVM Github Monorepo https://rev