On Nov 9, 2007, at 8:55 AM, Evan Cheng wrote: > Since I gave bad advice last time, how about one more advice? :-) > > Create a DenseMap mapping from target registers to dwarf numbers at > X86RegisterInfo ctor (or on demand the first time getDwarfRegNum is > called). So subsequent queries should be nice and fast.
Ick :) > Frankly I don't think the issue is worth spending this much time on. > Whatever works and maintainable is good. I'd prefer to use one switch statement form or the other. The compiler should be able to turn trivial switches like that into an array lookup. Note that it doesn't currently, but adding this shouldn't be hard. -Chris _______________________________________________ llvm-commits mailing list llvm-commits@cs.uiuc.edu http://lists.cs.uiuc.edu/mailman/listinfo/llvm-commits
