[PATCH] D48426: [clang-cl] Don't emit dllexport inline functions etc. from pch files (PR37801)

Nico Weber via Phabricator via cfe-commits cfe-commits at lists.llvm.org
Thu Jun 21 08:01:50 PDT 2018


thakis added a comment.

PCHs aren't compatible with themselves if only the compiler revision changes, so I'm not sure changing that field should be worse than a regular compiler revision update (which happens at every commit). But I don't know what this field is for. I don't remember any trouble from me changing it though, and if it was bad to change it I'd hope there'd be a comment right above the field telling us why. (If I had run into problems, I would've expected me to add a comment like this.)


https://reviews.llvm.org/D48426





More information about the cfe-commits mailing list