[Beignet] [PATCH] GBE: add thread_local for class static variable

Song, Ruiling ruiling.song at intel.com
Thu Oct 29 00:05:18 PDT 2015


You don't need to modify " static char ID ", and any llvm pass has the "static char ID", see llvm_to_gen.cpp, lots of llvm passes were invoked.
llvm will not modify the value of ID. It just use the address of ID to identify a pass.
The issue occurs because of " static map<CallInst*, PrintfSet::PrintfFmt*> printfs ", why don't change it to non-static member?
I think we should not use static member like 'printfs' in a FunctionPass.

Thanks!
Ruiling



More information about the Beignet mailing list