[PATCH] D124435: [X86] Always extend the integer parameters in callee
John McCall via Phabricator via cfe-commits
cfe-commits at lists.llvm.org
Wed Aug 3 11:42:12 PDT 2022
rjmccall added a comment.
I know what you're saying, but I don't think it matches any model of how programmers use command line flags. You're imagining that a programmer sits down and considers all of their flags deeply and holistically before touching any of them, and that's just not how these things go in actual build systems. Flags have a tendency to be set in separate places and therefore to drift. Someone setting this option is doing it because they have a concrete compatibility need, and the flag going from meaning "be compatible with GCC" to meaning "be compatible with old Clangs" based on the value of a separate flag is surprising and likely to cause bugs.
Repository:
rG LLVM Github Monorepo
CHANGES SINCE LAST ACTION
https://reviews.llvm.org/D124435/new/
https://reviews.llvm.org/D124435
More information about the cfe-commits
mailing list