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

Reply via email to