[PATCH] D130710: [ASTWriter] Provide capability to output a PCM/PCH file that does not write out information about its output path

Argyrios Kyrtzidis via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Fri Jul 29 14:28:43 PDT 2022


akyrtzi added a comment.

In D130710#3685436 <https://reviews.llvm.org/D130710#3685436>, @benlangmuir wrote:

> Is the functionality provided by `ORIGINAL_PCH_DIR` still useful for making it possible to move a PCH and its referenced headers?  It's not completely clear to me when this feature is used in practice.  It would be nice to remove it or change the default behaviour if possible, rather than require a new option, but I'm open to this approach if we think we can't change the default.

Given that there was a recent change <https://reviews.llvm.org/D124946> related to `ORIGINAL_PCH_DIR`, I'm reluctant to change the default at this point, I added a `FIXME` for following up to see if we can remove it or change the default later on.


Repository:
  rG LLVM Github Monorepo

CHANGES SINCE LAST ACTION
  https://reviews.llvm.org/D130710/new/

https://reviews.llvm.org/D130710



More information about the cfe-commits mailing list