[clang] [Preprocessor] Do not expand macros if the input is already preprocessed (PR #137665)

Jacob Lambert via cfe-commits cfe-commits at lists.llvm.org
Fri May 9 10:49:25 PDT 2025


Juan Manuel Martinez =?utf-8?q?CaamaƱo?= <juamarti at amd.com>
Message-ID:
In-Reply-To: <llvm.org/llvm/llvm-project/pull/137665 at github.com>


lamb-j wrote:

> module maps don't get preprocessed, so I don't think it matters what that is set to. For normal module.modulemap files `Preprocessed` is `false`. I think it's fine to remove the true and see if anything breaks, the preprocessor just skips over that section anyway.

Thanks for the pointer. @Bigcheese who should we solicit an approving review from? I'm not familiar enough with the implications of the change to feel confident approving

https://github.com/llvm/llvm-project/pull/137665


More information about the cfe-commits mailing list