Thanks for your advice, I will try what you said and test again. Thanks Xiuli Pan
-----Original Message----- From: Song, Ruiling Sent: Thursday, October 29, 2015 3:05 PM To: Pan, Xiuli <xiuli....@intel.com>; beignet@lists.freedesktop.org Cc: Pan, Xiuli <xiuli....@intel.com> Subject: RE: [Beignet] [PATCH] GBE: add thread_local for class static variable 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 _______________________________________________ Beignet mailing list Beignet@lists.freedesktop.org http://lists.freedesktop.org/mailman/listinfo/beignet