[patch][PR10367] Fix the design of GlobalAlias
Rafael EspĂndola
rafael.espindola at gmail.com
Wed May 14 09:29:05 PDT 2014
>> Yes. I started going that way, but the churn is so big that it is
>> probably better if we do that as a separate patch. I will send one
>> with just the renaming once these are done.
>
> Would that constitute a C API change? I'm thinking specifically about
> include/llvm-c/Core.h:
Yes. To be strict about the backwards compatibility we would need to have
LLVMIsAGlobalValue == LLVMIsAGlobalName
LLVMIsAGlobalValueV2 == checks for llvm::GlobalValue
which is pretty nasty. Not sure if we want that or if we can change
the meaning a bit.
In any case, it should be a nice topic for a followup thread :-)
Cheers,
Rafael
More information about the llvm-commits
mailing list