[PATCH] D23339: Don't import variadic functions

[Piotr Padlewski via llvm-commits]

Prazek marked an inline comment as done.

================
Comment at: lib/Bitcode/Reader/BitcodeReader.cpp:724
@@ -724,1 +723,3 @@
+/// Decode the flags for GlobalValue in the summary
+/// FIXME: remove Version because it is not used.
 static GlobalValueSummary::GVFlags getDecodedGVSummaryFlags(uint64_t RawFlags,
----------------
tejohnson wrote:
> The Version flag is there in case we ever add flags such that older versions of bitcode need to be handled specially. E.g. if you had left this as a positive IsViableToInline, we would probably want to bump the version and set that flag when we had an older version of bitcode (so we didn't completely stop importing when compiling with older bitcode). Since the reversed flag indicates when not to import (and it is a heuristic and not correctness), I don't think there is any need to bump/check the version.
> 
> The upshot is that the FIXME can be removed.
I see. maybe I should change Version => /*Version, not used right now*/ 


https://reviews.llvm.org/D23339