[lld] [lld][build] include Version.inc (PR #127946)

Fangrui Song via llvm-commits llvm-commits at lists.llvm.org
Wed Feb 19 22:06:23 PST 2025


MaskRay wrote:

I actually noticed that your previous lld patch did not include `.inc`. I thought that it was ok because that I don't find a `.inc` in the source tree (except include/lld/Common/BPSectionOrdererBase.inc, which does not define APIs).

I'd love to see more testing in the future to minimize the need for these kinds of fixes. Please update the description to say that this fixes #127123 

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


More information about the llvm-commits mailing list