[clang] Reset CodeGenOptions fields for clean module/PCH builds (PR #138256)

ayushpareek2003 wrote: > hmmm for `test case`, I mean a test case in clang, either a unit test or a > lit test, so we can understand the effect on users more clearly. > > And if you're talking about the general code policy, I'll suggest you to > https://discourse.llvm.org/ to make it more verb

[clang] Reset CodeGenOptions fields for clean module/PCH builds (PR #138256)

https://github.com/ayushpareek2003 updated https://github.com/llvm/llvm-project/pull/138256 Rate limit · GitHub body { background-color: #f6f8fa; color: #24292e; font-family: -apple-system,BlinkMacSystemFont,Segoe UI,Helvetica,Aria

[clang] Reset CodeGenOptions fields for clean module/PCH builds (PR #138256)

ayushpareek2003 wrote: > I feel `.clear()` should have a similar semantic with `= {};`. It will be > confusing for people to understand this. It will be helpful if you can > provide a test to show the difference. I’ve tried to demonstrate the difference between .clear() and = {}, not a formal

[clang] Reset CodeGenOptions fields for clean module/PCH builds (PR #138256)

ayushpareek2003 wrote: > Could you elaborate? Why `= {}` is better than `.clear()` ? according to my understanding, Clang stores some build-specific data in CodeGenOptions like file paths and debug info. When generating modules or precompiled headers, this data can make the output different fr

[clang] Reset CodeGenOptions fields for clean module/PCH builds (PR #138256)

https://github.com/ayushpareek2003 created https://github.com/llvm/llvm-project/pull/138256 This change resets certain CodeGenOptions fields using = {} instead of .clear(), to clean up build-specific data when generating modules or PCH files. It helps make the output more consistent and easier

[clang] Optimize Module Dependency Handling for Efficient Memory Usage (PR #133524)

https://github.com/ayushpareek2003 created https://github.com/llvm/llvm-project/pull/133524 -Optimized addModuleFiles functions for both CompilerInvocation and CowCompilerInvocation to reduce redundant function calls and improve efficiency -Introduced memory preallocation using reserve() when

[clang] Optimize Module Dependency Handling for Efficient Memory Usage (PR #132287)

https://github.com/ayushpareek2003 closed https://github.com/llvm/llvm-project/pull/132287 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] refactor OffloadAction constructor for memory efficiency & cleaner access (PR #134447)

https://github.com/ayushpareek2003 created https://github.com/llvm/llvm-project/pull/134447 Avoided repeated DeviceDependences getter calls by using const auto & references No functional changes; pure memory and readability optimization >From 2689233873e3c0d998bbfb7bb240d39d1654a973 Mon Sep 17

[clang] Optimize Module Dependency Handling for Efficient Memory Usage (PR #132287)

ayushpareek2003 wrote: @Bigcheese Please have a look https://github.com/llvm/llvm-project/pull/132287 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] Optimize Module Dependency Handling for Efficient Memory Usage (PR #132294)

https://github.com/ayushpareek2003 created https://github.com/llvm/llvm-project/pull/132294 -Optimized addModuleFiles functions for both CompilerInvocation and CowCompilerInvocation to reduce redundant function calls and improve efficiency -Introduced memory preallocation using reserve() when

[clang] Relative Path Handling for Cross-Platform Compatibility (PR #133526)

https://github.com/ayushpareek2003 created https://github.com/llvm/llvm-project/pull/133526 Both Unix (./, ../) and Windows (.\, ..\) path formats are handled properly >From 7c70f0c2bf45b34c1134820a21ec169777bce7bf Mon Sep 17 00:00:00 2001 From: Ayush Pareek Date: Sat, 29 Mar 2025 03:25:39 +05

[clang] Optimize Module Dependency Handling for Efficient Memory Usage (PR #132638)

https://github.com/ayushpareek2003 updated https://github.com/llvm/llvm-project/pull/132638 >From ba67caab088be29018c17bfe47e3064db357788d Mon Sep 17 00:00:00 2001 From: Ayush Pareek Date: Mon, 24 Mar 2025 03:33:31 +0530 Subject: [PATCH] Optimize Module Dependency Handling for Efficient Memory

[clang] Optimize Module Dependency Handling for Efficient Memory Usage (PR #132638)

ayushpareek2003 wrote: @Bigcheese sir please have a look https://github.com/llvm/llvm-project/pull/132638 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] Optimize Module Dependency Handling for Efficient Memory Usage (PR #132294)

https://github.com/ayushpareek2003 updated https://github.com/llvm/llvm-project/pull/132294 >From 53a31ec6ed3d9ace8e7d822c2b2de4f6f58ece8d Mon Sep 17 00:00:00 2001 From: Ayush Pareek Date: Fri, 21 Mar 2025 03:44:05 +0530 Subject: [PATCH 1/4] Optimize Module Dependency Handling for Efficient Mem

[clang] Optimize Module Dependency Handling for Efficient Memory Usage (PR #132294)

https://github.com/ayushpareek2003 updated https://github.com/llvm/llvm-project/pull/132294 >From 53a31ec6ed3d9ace8e7d822c2b2de4f6f58ece8d Mon Sep 17 00:00:00 2001 From: Ayush Pareek Date: Fri, 21 Mar 2025 03:44:05 +0530 Subject: [PATCH] Optimize Module Dependency Handling for Efficient Memory

[clang] Optimize Module Dependency Handling for Efficient Memory Usage (PR #132638)

https://github.com/ayushpareek2003 created https://github.com/llvm/llvm-project/pull/132638 -Optimized addModuleFiles functions for both CompilerInvocation and CowCompilerInvocation to reduce redundant function calls and improve efficiency -Introduced memory preallocation using reserve() when

[clang] [llvm] Optimize Module Dependency Handling for Efficient Memory Usage (PR #132294)

https://github.com/ayushpareek2003 updated https://github.com/llvm/llvm-project/pull/132294 >From 53a31ec6ed3d9ace8e7d822c2b2de4f6f58ece8d Mon Sep 17 00:00:00 2001 From: Ayush Pareek Date: Fri, 21 Mar 2025 03:44:05 +0530 Subject: [PATCH 1/3] Optimize Module Dependency Handling for Efficient Mem

[clang] [llvm] Optimize Module Dependency Handling for Efficient Memory Usage (PR #132294)

https://github.com/ayushpareek2003 closed https://github.com/llvm/llvm-project/pull/132294 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] [llvm] Optimize Module Dependency Handling for Efficient Memory Usage (PR #132294)

https://github.com/ayushpareek2003 updated https://github.com/llvm/llvm-project/pull/132294 >From 53a31ec6ed3d9ace8e7d822c2b2de4f6f58ece8d Mon Sep 17 00:00:00 2001 From: Ayush Pareek Date: Fri, 21 Mar 2025 03:44:05 +0530 Subject: [PATCH 1/2] Optimize Module Dependency Handling for Efficient Mem

[clang] Retry for Unique Path Creation (PR #132640)

https://github.com/ayushpareek2003 created https://github.com/llvm/llvm-project/pull/132640 At line 754, added a FOR loop for retrying until we get a unique path >From 3927b90a26ea1bd04fe7e55a6d21e7d27ce68839 Mon Sep 17 00:00:00 2001 From: Ayush Pareek Date: Mon, 24 Mar 2025 03:45:31 +0530 Sub

[clang] Optimize Module Dependency Handling for Efficient Memory Usage (PR #132294)

ayushpareek2003 wrote: @Bigcheese sir please have a look https://github.com/llvm/llvm-project/pull/132294 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] Optimize Module Dependency Handling for Efficient Memory Usage (PR #132287)

https://github.com/ayushpareek2003 created https://github.com/llvm/llvm-project/pull/132287 Optimized addModuleFiles functions for both CompilerInvocation and CowCompilerInvocation to reduce redundant function calls and improve efficiency Introduced memory preallocation using reserve() when ea

[clang] Optimize Cache Insertion with try_emplace for Reduced Lookups (PR #131402)

https://github.com/ayushpareek2003 commented: Removed unnecessary spaces https://github.com/llvm/llvm-project/pull/131402 ___ cfe-commits mailing list cfe-commits@lists.llvm.org https://lists.llvm.org/cgi-bin/mailman/listinfo/cfe-commits

[clang] Optimize Cache Insertion with try_emplace for Reduced Lookups (PR #131402)

https://github.com/ayushpareek2003 updated https://github.com/llvm/llvm-project/pull/131402 >From 28f6d8b6677e32f45f5fa55c7c73df5a841d7127 Mon Sep 17 00:00:00 2001 From: Ayush Pareek Date: Sat, 15 Mar 2025 03:43:18 +0530 Subject: [PATCH 1/2] Optimize Cache Insertion with try_emplace for Reduced

[clang] Optimize Cache Insertion with try_emplace for Reduced Lookups (PR #131402)

https://github.com/ayushpareek2003 updated https://github.com/llvm/llvm-project/pull/131402 >From 28f6d8b6677e32f45f5fa55c7c73df5a841d7127 Mon Sep 17 00:00:00 2001 From: Ayush Pareek Date: Sat, 15 Mar 2025 03:43:18 +0530 Subject: [PATCH] Optimize Cache Insertion with try_emplace for Reduced Loo

[clang] Remove redundant assertion & fix ClearStatName error (PR #130667)

https://github.com/ayushpareek2003 created https://github.com/llvm/llvm-project/pull/130667 Issue: Calling clearStatName() on an error object The function clearStatName() is called inside constructors before checking whether MaybeStat contains an error. If MaybeStat is an error, calling copyW

[clang] Optimize Cache Insertion with try_emplace for Reduced Lookups (PR #131402)

https://github.com/ayushpareek2003 created https://github.com/llvm/llvm-project/pull/131402 for the functions- insertEntryForFilename() , insertRealPathForFilename() Replaced `Cache.insert()` with `Cache.try_emplace()` to reduce redundant lookups Improved efficiency by avoiding unnecessary co

[clang] Remove redundant assertion & fix ClearStatName error (PR #130667)

@@ -126,7 +126,8 @@ class CachedFileSystemEntry { private: void clearStatName() { -if (MaybeStat) + +if (MaybeStat && MaybeStat->getName().empty()) //If MaybeStat is an error, calling copyWithNewName() can cause undefined behavior ayushparee

[clang] Remove redundant assertion & fix ClearStatName error (PR #130667)

https://github.com/ayushpareek2003 updated https://github.com/llvm/llvm-project/pull/130667 >From 94b213ad0edf8295451cdb315093cd73923714bb Mon Sep 17 00:00:00 2001 From: Ayush Pareek Date: Tue, 11 Mar 2025 02:23:49 +0530 Subject: [PATCH 1/2] Update DependencyScanningFilesystem.h Issue: Calling

[clang] Remove redundant assertion & fix ClearStatName error (PR #130667)

@@ -93,7 +93,7 @@ class CachedFileSystemEntry { getDirectiveTokens() const { assert(!isError() && "error"); assert(!isDirectory() && "not a file"); -assert(Contents && "contents not initialized"); +// Since isError() and isDirectory() imply that Contents is nu